Skip to content

Commit 7b153b4

Browse files
committed
[compiler-rt] Partially revert 8bd2722
Don't normalize arm architecture names; doing that loses the ability to pick the right implementation of builtins for each architecture variant. When building compiler-rt builtins as part of a runtimes build, builtins for multiple armv* variants could be built in the same directory, and with the simplified architecture name, they'd all be built in the same directory, overlapping each other.
1 parent 2fe85dd commit 7b153b4

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

compiler-rt/cmake/Modules/CompilerRTUtils.cmake

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -332,13 +332,6 @@ macro(construct_compiler_rt_default_triple)
332332
if("${COMPILER_RT_DEFAULT_TARGET_ARCH}" MATCHES "^i.86$")
333333
# Android uses i686, but that's remapped at a later stage.
334334
set(COMPILER_RT_DEFAULT_TARGET_ARCH "i386")
335-
elseif ("${COMPILER_RT_DEFAULT_TARGET_ARCH}" MATCHES "^arm" AND
336-
NOT "${COMPILER_RT_DEFAULT_TARGET_ARCH}" MATCHES "^arm64")
337-
if ("${COMPILER_RT_DEFAULT_TARGET_TRIPLE}" MATCHES ".*hf$")
338-
set(COMPILER_RT_DEFAULT_TARGET_ARCH "armhf")
339-
else()
340-
set(COMPILER_RT_DEFAULT_TARGET_ARCH "arm")
341-
endif()
342335
endif()
343336

344337
# Determine if test target triple is specified explicitly, and doesn't match the

0 commit comments

Comments
 (0)