[llvm] [Release] Install compiler-rt builtins during Phase 1 on AIX (PR #81485)

Hubert Tong via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 13 15:24:23 PST 2024


================
@@ -534,7 +534,7 @@ function build_llvmCore() {
       # compiler-rt builtins is needed on AIX to have a functional Phase 1 clang.
       if [ "$System" = "AIX" -o "$Phase" != "1" ]; then
         BuildTarget="$BuildTarget runtimes"
-        InstallTarget="$InstallTarget install-runtimes"
+        InstallTarget="$InstallTarget install-runtimes install-builtins"
----------------
hubert-reinterpretcast wrote:

> This was another mistake. You are correct there is a bug in the current logic.

@tstellar:
Was the intent for only AIX to have the extra targets added on Phase 1 and 2?
(I would be surprised if it was needed for only Phase 1 and not Phase 2).


https://github.com/llvm/llvm-project/pull/81485


More information about the llvm-commits mailing list