[clang] [compiler-rt] [llvm] Get the linker version and pass the it to compiler-rt tests on Darwin. (PR #86220)
Julian Lettner via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 22 11:29:03 PDT 2024
================
@@ -346,20 +347,7 @@ endif ()
# Determine HOST_LINK_VERSION on Darwin.
set(HOST_LINK_VERSION)
if (APPLE AND NOT CMAKE_LINKER MATCHES ".*lld.*")
- set(LD_V_OUTPUT)
- execute_process(
- COMMAND sh -c "${CMAKE_LINKER} -v 2>&1 | head -1"
- RESULT_VARIABLE HAD_ERROR
- OUTPUT_VARIABLE LD_V_OUTPUT
- )
- if (HAD_ERROR)
- message(FATAL_ERROR "${CMAKE_LINKER} failed with status ${HAD_ERROR}")
- endif()
- if ("${LD_V_OUTPUT}" MATCHES ".*ld64-([0-9.]+).*")
- string(REGEX REPLACE ".*ld64-([0-9.]+).*" "\\1" HOST_LINK_VERSION ${LD_V_OUTPUT})
- elseif ("${LD_V_OUTPUT}" MATCHES "[^0-9]*([0-9.]+).*")
- string(REGEX REPLACE "[^0-9]*([0-9.]+).*" "\\1" HOST_LINK_VERSION ${LD_V_OUTPUT})
- endif()
+ get_darwin_linker_version()
----------------
yln wrote:
- [ ] Would it be possible to "declare" where the value is stored, i.e., a CMAKE output parameter? So the relationship between `HOST_LINK_VERSION` and `get_darwin_linker_version()` is explicit (and other variable names can be used)?
```suggestion
get_darwin_linker_version(HOST_LINK_VERSION)
```
https://github.com/llvm/llvm-project/pull/86220
More information about the cfe-commits
mailing list