Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ITK_x86 failed to run test on Asan mode #4448

Open
WillaCao opened this issue Feb 5, 2024 · 6 comments
Open

ITK_x86 failed to run test on Asan mode #4448

WillaCao opened this issue Feb 5, 2024 · 6 comments
Labels
type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances

Comments

@WillaCao
Copy link

WillaCao commented Feb 5, 2024

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

@WillaCao WillaCao added the type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances label Feb 5, 2024
Copy link

github-actions bot commented Feb 5, 2024

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
Copy link
Member

dzenanz commented Feb 5, 2024

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

@N-Dekker
Copy link
Contributor

N-Dekker commented Feb 6, 2024

@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
Copy link
Author

WillaCao commented Feb 7, 2024

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

@N-Dekker
Copy link
Contributor

N-Dekker commented Feb 7, 2024

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
Copy link
Author

WillaCao commented Feb 8, 2024

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances
Projects
None yet
Development

No branches or pull requests

3 participants