[llvm] [LLVM][runtimes] Prepopulate `LLVM_BUILTIN_TARGETS` with runtimes values (PR #95554)
Saleem Abdulrasool via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 25 10:06:03 PDT 2024
================
@@ -138,37 +142,44 @@ endfunction()
# before the just-built compiler can pass the configuration tests.
get_compiler_rt_path(compiler_rt_path)
if(compiler_rt_path)
- if(NOT LLVM_BUILTIN_TARGETS)
+ # If the user did not specify the targets infer them from the runtimes.
+ set(builtin_targets ${LLVM_BUILTIN_TARGETS})
+ if(NOT builtin_targets)
+ if("compiler-rt" IN_LIST LLVM_ENABLE_RUNTIMES)
+ list(APPEND builtin_targets "default")
+ endif()
----------------
compnerd wrote:
I'm not sure that applying DRY here is worth it. I'd rather just duplicate the `builtin_default_target` invocation and remove the string handling for "default".
https://github.com/llvm/llvm-project/pull/95554
More information about the llvm-commits
mailing list