search

InsightSoftwareConsortium / ITK

Insight Toolkit (ITK) -- Official Repository. ITK builds on a proven, spatially-oriented architecture for processing, segmentation, and registration of scientific images in two, three, or more dimensions.
https://itk.org
Apache License 2.0
1.42k stars 664 forks source link

ITK_x86 failed to run test on Asan mode #4448

Open WillaCao opened 9 months ago

WillaCao commented 9 months ago

Description

The issue only repro on X86 Asan mode. Could you please help to take a look at this? Thank you.

Steps to Reproduce

  1. git clone https://github.com/InsightSoftwareConsortium/ITK F:\gitP\InsightSoftwareConsortium\ITK
  2. cd F:\gitP\InsightSoftwareConsortium\ITK
  3. git rev-parse --git-dir
  4. git clean -xdf
  5. git fetch --recurse-submodules=no --force
  6. git reset --hard 12328e1
  7. git remote prune origin
  8. set VSCMD_SKIP_SENDTELEMETRY=1 & "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -host_arch=x86 -arch=x86 & set CL= /fsanitize=address /GS- /wd5072 & set LINK= /InferASanLibs /incremental:no /debug
  9. cd F:\gitP\InsightSoftwareConsortium\ITK\build_x86
  10. cmake -G "Visual Studio 17 2022" -A Win32 -DCMAKE_SYSTEM_VERSION=10.0.22621.0 -DBUILD_SHARED_LIBS=OFF -DBUILD_TESTING=ON -DBUILD_EXAMPLES=OFF ..
  11. set CL=/D_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR %CL%
  12. set CL=/D_SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING %CL%
  13. msbuild /m /p:Platform=Win32 /p:Configuration=Release All_BUILD.vcxproj /t:Rebuild
  14. ctest --build-config Release --output-on-failure -R "itkImportContainerTest|itkNormalizedCorrelationImageFilterTest|itkInvertDisplacementFieldImageFilterTest|itkInterpolateImageFilterTest|itkWarpImageFilterTest|itkWarpVectorImageFilterTest|itkExpectationBasedPointSetMetricTest|itkJointHistogramMutualInformationImageToImageRegistrationTest|itkJointHistogramMutualInformationImageToImageRegistrationTest2|itkMeanSquaresImageToImageMetricv4RegistrationTest|itkDemonsImageToImageMetricv4RegistrationTest|itkDemonsImageToImageMetricv4RegistrationTest2|itkDemonsImageToImageMetricv4RegistrationTest3|itkExpectationBasedPointSetMetricRegistrationTest|itkObjectToObjectMultiMetricv4Test|itkMeanSquaresImageToImageMetricv4VectorRegistrationTest|itkRegistrationParameterScalesFromPhysicalShiftTest|itkRegistrationParameterScalesFromJacobianTest|itkRegistrationParameterScalesFromIndexShiftTest|itkPathFunctionsTest|itkChainCodeToFourierSeriesPathFilterTest|itkExtractOrthogonalSwath2DImageFilterTest|itkOrthogonalSwath2DPathFilterTest|itkSimpleImageRegistrationTestDouble|itkSimpleImageRegistrationTestFloat|itkSimpleImageRegistrationTestWithMaskAndSamplingDouble|itkSimpleImageRegistrationTestWithMaskAndSamplingFloat|itkQuasiNewtonOptimizerv4RegistrationTest1|itkQuasiNewtonOptimizerv4RegistrationTest2|itkQuasiNewtonOptimizerv4RegistrationTest3|itkBSplineImageRegistrationTest"

    Expected behavior

Run tests all pass

Actual behavior

error_log.txt

github-actions[bot] commented 9 months ago

Thank you for contributing an issue! 🙏

Welcome to the ITK community! 🤗👋☀️

We are glad you are here and appreciate your contribution. Please keep in mind our community participation guidelines. 📜 Also, please check existing open issues and consider discussion on the ITK Discourse. 📖

This is an automatic message. Allow for time for the ITK community to be able to read the issue and comment on it.

dzenanz commented 9 months ago

@issakomi @N-Dekker can either of you look into this?

N-Dekker commented 9 months ago

@WillaCao Thank you for reporting this issue. Do I understand correctly that it is specific to compilation for 32-bit? If so, do you have a specific reason to build for 32-bit, rather than for 64-bit? Just for my understanding!

WillaCao commented 9 months ago

@N-Dekker I compiled with both 32-bit and 64-bit, but 64-bit passed.

N-Dekker commented 9 months ago

I compiled with both 32-bit and 64-bit, but 64-bit passed.

Thanks @WillaCao I'm glad to hear that 64-bit still passed 👍 Personally I haven't been building 32-bits applications on Windows for a long time. Is 32-bit support on Windows still important to you?

WillaCao commented 9 months ago

@N-Dekker We are using an open-source project to test the performance of the MSVC compiler, so we need to compile for both 32-bit and 64-bit.