Merge topic 'binutils-strip-macos'

ac2562afb3 BinUtils: Avoid llvm-strip on Apple platforms

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !8372
This commit is contained in:
Brad King
2023-03-31 12:01:00 +00:00
committed by Kitware Robot
+9 -1
View File
@@ -186,7 +186,15 @@ else()
list(PREPEND _CMAKE_RANLIB_NAMES "llvm-ranlib")
if("${CMAKE_${_CMAKE_PROCESSING_LANGUAGE}_COMPILER_VERSION}" VERSION_GREATER_EQUAL 11)
# llvm-strip versions prior to 11 require additional flags we do not yet add.
list(PREPEND _CMAKE_STRIP_NAMES "llvm-strip")
if(APPLE)
# llvm-strip does not seem to support chained fixup format correctly.
# FIXME(#23333): We still need to consider 'llvm-strip' as a fallback
# because the 'APPLE' definition may be based on the host in this context,
# and a cross-compiling toolchain may not have 'strip'.
list(APPEND _CMAKE_STRIP_NAMES "llvm-strip")
else()
list(PREPEND _CMAKE_STRIP_NAMES "llvm-strip")
endif()
endif()
list(PREPEND _CMAKE_NM_NAMES "llvm-nm")
if("${CMAKE_${_CMAKE_PROCESSING_LANGUAGE}_COMPILER_VERSION}" VERSION_GREATER_EQUAL 9)