[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