[libcxx-commits] [libcxx] c64f10b - [clang] Build the ToT Clang in Release mode in the Clang/libcxx CI pipeline

[Louis Dionne via libcxx-commits]

Author: Louis Dionne
Date: 2023-05-02T17:15:14-04:00
New Revision: c64f10bfe20308ebc7d5d18912cd0ba82a44eaa1

URL: https://github.com/llvm/llvm-project/commit/c64f10bfe20308ebc7d5d18912cd0ba82a44eaa1
DIFF: https://github.com/llvm/llvm-project/commit/c64f10bfe20308ebc7d5d18912cd0ba82a44eaa1.diff

LOG: [clang] Build the ToT Clang in Release mode in the Clang/libcxx CI pipeline

This is an attempt to reduce the insane amount of network we use when
uploading Clang binaries to Buildkite for other jobs to use. We think
that removing debug information will result in much smaller binaries.

Added: 
    

Modified: 
    libcxx/utils/ci/buildkite-pipeline-clang.yml

Removed: 
    


################################################################################
diff  --git a/libcxx/utils/ci/buildkite-pipeline-clang.yml b/libcxx/utils/ci/buildkite-pipeline-clang.yml
index e9eefa85cf0e4..3e40c2bc0fef3 100644
--- a/libcxx/utils/ci/buildkite-pipeline-clang.yml
+++ b/libcxx/utils/ci/buildkite-pipeline-clang.yml
@@ -19,7 +19,10 @@ steps:
   - label: "Building clang"
     commands:
       - "mkdir install"
-      - "cmake -S llvm -B build -G Ninja -DCMAKE_BUILD_TYPE=RelWithDebInfo -DCMAKE_INSTALL_PREFIX=install -DLLVM_ENABLE_PROJECTS=\"clang;compiler-rt\""
+      # We use Release here to avoid including debug information. Otherwise, the clang binary is very large, which
+      # is problematic because we need to upload the artifacts for other jobs to use. This may seem like nothing,
+      # but with the number of jobs we run daily, this can result in thousands of GB of network I/O.
+      - "cmake -S llvm -B build -G Ninja -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=install -DLLVM_ENABLE_PROJECTS=\"clang;compiler-rt\""
       - "ninja -C build install-clang install-clang-resource-headers"
       - "buildkite-agent artifact upload --debug 'install/**/*'"
     env: