Re: [boost] utility/value_init.hpp: VC 7.1 ICE & workaround

Hi Niels,
Thanks for reporting the ICE caused by the MSVC workaround I committed last week! My apologizes! I didn't know about this issue. The ICE did not occur when doing the unit tests (value_init_test.cpp).
The Boost.Python unit tests are also fine. Even most of our real code compiles with the svn head as is. Only about 20 of our 60 or so extensions fail with the ICE.
Your patch:
Simply inlining get() didn't work. The ICE only went away after I changed (*static_cast<T*>(ptr)).T::~T() to static_cast<T*>(ptr)->T::~T()
Does the ICE also occur when using the destructor of the other base class, const_T_base? If so, it would need a similar patch.
I don't know. I just played around for a couple of minutes until the ICE went away.
I will take a closer look later today, or otherwise tomorrow. And I'll contact Fernando Cacciola, the creator and maintainer of the file.
If you want to reproduce the problem, I could point you to the sources. But I think it would be a bit too much effort since the problem is resolved so easily, and it is just a workaround in a workaround... Ralf ____________________________________________________________________________________ Be a better Heartthrob. Get better relationship answers from someone who knows. Yahoo! Answers - Check it out. http://answers.yahoo.com/dir/?link=list&sid=396545433

The ICE did not occur when doing the unit tests (value_init_test.cpp).
Ralf W. Grosse-Kunstleve wrote:
The Boost.Python unit tests are also fine. Even most of our real code compiles with the svn head as is. Only about 20 of our 60 or so extensions fail with the ICE.
Your patch looks perfectly fine to me, but I'm still curious to know in what particular circumstances my code causes a compiler crash.
If you want to reproduce the problem, I could point you to the sources.
Please do :-) Maybe the specific scenario should be added to the unit test!
But I think it would be a bit too much effort since the problem is resolved so easily
It was my very first Boost commit, and Fernando and I have put a lot of effort in making it MSVC-compliant, so I find it rather shocking that it causes an ICE! Kind regards, Niels -- Niels Dekker http://www.xs4all.nl/~nd/dekkerware Scientific programmer at LKEB, Leiden University Medical Center
participants (2)
-
Niels Dekker - mail address until 2008-12-31
-
Ralf W. Grosse-Kunstleve