[ITK] Win64 Visual Studio 12 segfaults

Wed Feb 4 08:53:59 EST 2015

Thank you Marius and Niels,

Outstanding work. The VS12 dashboard machine has all it's tests passing last night!

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

Brad

________________________________________
From: Niels Dekker [niels-xtk at xs4all.nl]
Sent: Monday, February 02, 2015 9:45 AM
To: community at itk.org
Subject: Re: [ITK] Win64 Visual Studio 12 segfaults

Marius Staring and I have just placed three patches onto Gerrit, that
should fix both the segfaults and the internal compiler errors of Visual
Studio 2013 Win64. Each of the three patches can be committed onto the ITK
GIT master independently, but I think the segfaults patch have the highest
priority. Fortunately, it already has a "+1", thank you, Bradley!

  "BUG: workaround for 64bit VS2013 Release segfaults"
  http://review.source.kitware.com/#/c/19080/

  "COMP: workaround for 64bit VS2013 Release ICE"
  http://review.source.kitware.com/#/c/19081/

  "WIP: COMP: workaround for 64bit VS2013 Release ICEs"
  http://review.source.kitware.com/#/c/19082/

As you can see, there appear two ways to fix the ICEs (internal compiler
errors): either by some small modification of a 'for' loop (currently only
needed in "itkQuasiNewtonOptimizerv4.hxx", review 19081), or by avoiding
narrowing integer conversions, from 64-bits to 32-bits, when calling
vnl_vector::operator[] (review 19082).

The change of the parameter-type of vnl_vector::operator[] may have
implications outside of ITK as well, and it could be part of a larger
effort to improve 64-bits support, so we marked it "WIP"
(work-in-progress). However, personally I think the commit itself (review
19082) is already a step in the right direction, and clearly it already
avoids those ICE's.

Kind regards, Niels
--
Niels Dekker
Scientific programmer
LKEB, Leiden University Medical Center

_______________________________________________
Community mailing list
Community at itk.org
http://public.kitware.com/mailman/listinfo/community