[ITK] Win64 Visual Studio 12 segfaults

[Bradley Lowekamp]

Hello,

I also have a SimpleITK build with Visual Studio 2013 (VS12) for SimpleITK for Win64 build against the ITK release branch:

https://open.cdash.org/buildSummary.php?buildid=3671881

It is down to 36 failures from 1287.  The CollidingFronts and FastMarchingUpwindGradient are SimpleITK issues.

What's interesting is that there no report internal compiler error, with 64-bit ids enabled. We a bunch of Deconvolution filters failing put nothing else.

Well this builds confidence in correct code being generated, I still would not give the generated code away to anyone and expect it to work 100%.

I think we should likely add a CMake warning for this problematic compiler and architecture.

Thanks,
Brad

On Jan 29, 2015, at 10:27 AM, Matt McCormick <matt.mccormick at kitware.com> wrote:

> Hi Niels,
> 
> That is excellent!
> 
> I'll wait until we have the patches merged into the release branch
> before releasing 4.7.1.  We can then explain the situation in the
> release announcement to improve awareness.  I don't think an issue
> tracker issue will help.
> 
> Thanks,
> Matt
> 
> On Thu, Jan 29, 2015 at 10:16 AM, Niels Dekker <niels-xtk at xs4all.nl> wrote:
>> Good news: We just did an Experimental Build, including ITK_USE_64BITS and
>> Module_ITKReview, with our (local) patches, and it has "100% tests passed,
>> 0 tests failed out of 2517"! Build Name: "Win64-VS2013-Release-64BITS_IDS
>> patched", https://open.cdash.org/buildSummary.php?buildid=3673003
>> 
>> In order to make more people aware of these Win64 compiler bugs, as well
>> as our workarounds, do you think it would be useful to add an "issue" (or
>> two) to https://issues.itk.org/jira/secure/Dashboard.jspa ?
>> 
>> Kind regards, Niels
>> 
>> Matt McCormick wrote on Jan 27, 2015:
>>> Thanks for the update!  Looking forward to the patches!
>>> 
>>> On Tue, Jan 27, 2015 at 2:54 PM, Niels Dekker <niels-xtk at xs4all.nl> wrote:
>>>> Status update: Floris Berendsen and I are currently testing a few
>>>> workarounds for the build errors and test failures that appeared at the
>>>> Dashboard for Win64-VS12-Release-Shared after setting
>>>> ITK_USE_64BITS_IDS:
>>>> https://open.cdash.org/buildSummary.php?buildid=3670301
>>>> 
>>>> Next week, we expect to be able to submit a patch to
>>>> http://review.source.kitware.com, with the help of our colleague Marius
>>>> Staring.  Hope it's still in time for the next ITK release  :-)
>>>> 
>>>> Furthermore, you may have seen that both compiler bugs have been
>>>> confirmed by Microsoft by now, and that Gratian Lup (Microsoft)
>>>> commented that these two bugs have been been fixed for the Visual
>>>> Studio 2015:
>>>> 
>>>> "Compiler generates divpd instruction on non-aligned data",
>>>> https://connect.microsoft.com/VisualStudio/feedbackdetail/view/1060166
>>>> 
>>>> "VS2013 x64 Release (/O2) fatal error C1001 on narrowing integer
>>>> conversions...",
>>>> https://connect.microsoft.com/VisualStudio/feedbackdetail/view/1093610
>>>> 
>>>> Matt, thanks for your vote on the bug report  :-)
>>>> 
>>>> 
>>>> Kind regards, Niels
>>>> --
>>>> Niels Dekker
>>>> Scientific programmer at LKEB, Leiden University Medical Center
>> 
>> 
>> _______________________________________________
>> Community mailing list
>> Community at itk.org
>> http://public.kitware.com/mailman/listinfo/community
> _______________________________________________
> Community mailing list
> Community at itk.org
> http://public.kitware.com/mailman/listinfo/community