Skip to content

Commit 6559330

Browse files
authored
[compiler-rt] Only include asan on x86 architectures on Windows (llvm#137173)
This avoids building asan when targeting Windows on armv7 or aarch64. It is possible to build asan successfully for those configurations (since 5ea9dd8 and 0c39113), but asan isn't functional there. This change skips building asan for targets other than x86_32 and x86_64. By excluding asan from the build, we fix the "check-ubsan" target for armv7 and aarch64 Windows. If asan is included in the build, an ubsan-asan configuration gets added to the tests, and as asan isn't functional for these targets, it produces a lot of test failures even when just trying to run "check-ubsan".
1 parent 34660eb commit 6559330

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

compiler-rt/cmake/Modules/AllSupportedArchDefs.cmake

+4
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,10 @@ else()
4444
set(OS_NAME "${CMAKE_SYSTEM_NAME}")
4545
endif()
4646

47+
if (OS_NAME MATCHES "Windows")
48+
set(ALL_ASAN_SUPPORTED_ARCH ${X86} ${X86_64})
49+
endif()
50+
4751
if(OS_NAME MATCHES "Linux")
4852
set(ALL_FUZZER_SUPPORTED_ARCH ${X86} ${X86_64} ${ARM32} ${ARM64} ${S390X}
4953
${RISCV64} ${LOONGARCH64})

0 commit comments

Comments
 (0)