Skip to content

Commit 25fa127

Browse files
committed
[compiler-rt] Only include asan on x86 architectures on Windows
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 f218cd2 commit 25fa127

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

compiler-rt/cmake/Modules/AllSupportedArchDefs.cmake

Lines changed: 4 additions & 0 deletions
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)