[llvm] [LLVM][runtimes] Prepopulate `LLVM_BUILTIN_TARGETS` with runtimes values (PR #95554)
Petr Hosek via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 12 02:23:43 PDT 2024
================
@@ -17,7 +17,11 @@ foreach(proj ${LLVM_ENABLE_RUNTIMES})
endforeach()
function(get_compiler_rt_path path)
- foreach(entry ${runtimes})
+ set(all_runtimes ${runtimes})
+ foreach(name ${LLVM_RUNTIME_TARGETS})
+ list(APPEND all_runtimes RUNTIMES_${name}_LLVM_ENABLE_RUNTIMES)
----------------
petrhosek wrote:
I think you need to evaluate the variable otherwise you're just going to add `RUNTIMES_${name}_LLVM_ENABLE_RUNTIMES` as an element to the list.
```suggestion
list(APPEND all_runtimes ${RUNTIMES_${name}_LLVM_ENABLE_RUNTIMES})
```
https://github.com/llvm/llvm-project/pull/95554
More information about the llvm-commits
mailing list