[llvm] [Release] Build default target during Phase 1 and 2 on AIX (PR #81009)

via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 7 08:22:15 PST 2024


https://github.com/azhan92 created https://github.com/llvm/llvm-project/pull/81009

Commit [`fcae8c`](https://github.com/llvm/llvm-project/pull/72703/commits/fcae8ce77e2256920459fb4cf428a212d6560b68) seems to be skipping the compiler-rt build  during Phase 1 which results in a non-functional Phase 1 clang on AIX

>From c8271ec50fc9a5096e7859f63dae633f76eb469d Mon Sep 17 00:00:00 2001
From: Alison Zhang <alisonxzhang at gmail.com>
Date: Wed, 7 Feb 2024 15:48:01 +0000
Subject: [PATCH] compiler-rt not being built during phase 1

---
 llvm/utils/release/test-release.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/llvm/utils/release/test-release.sh b/llvm/utils/release/test-release.sh
index 5b1945df47d24a..f18d15942eefc5 100755
--- a/llvm/utils/release/test-release.sh
+++ b/llvm/utils/release/test-release.sh
@@ -528,11 +528,11 @@ function build_llvmCore() {
     LitVerbose="-v"
 
     InstallTarget="install"
-    if [ "$Phase" -lt "3" ]; then
+    # compiler-rt builtins is needed on AIX to have a functional Phase 1 clang.
+    if [ "$Phase" -lt "3" ] && [ "$System" != "AIX" ]; then
       BuildTarget="clang"
       InstallTarget="install-clang install-clang-resource-headers"
-      # compiler-rt builtins is needed on AIX to have a functional Phase 1 clang.
-      if [ "$System" = "AIX" -o "$Phase" != "1" ]; then
+      if [ "$Phase" != "1" ]; then
         BuildTarget="$BuildTarget runtimes"
         InstallTarget="$InstallTarget install-runtimes"
       fi



More information about the llvm-commits mailing list