[llvm] [ci] Use check-compiler-rt target for testing compiler-rt (PR #111515)

via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 8 03:16:13 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-github-workflow

Author: David Spickett (DavidSpickett)

<details>
<summary>Changes</summary>

Instead of "check-all" which leads to us running some tests twice if there are other "check-..." targets. For example on one of my PRs this script produced:
```
commands:
  - './.ci/monolithic-linux.sh "clang;clang;lld;clang-tools-extra;compiler-rt;llvm" "check-all check-clang check-clang-tools" "libcxx;libcxxabi;libunwind" "check-cxx check-cxxabi check-unwind"'
  commands:
  - 'C:\BuildTools\Common7\Tools\VsDevCmd.bat -arch=amd64 -host_arch=amd64'
  - 'bash .ci/monolithic-windows.sh "clang;clang-tools-extra;llvm" "check-clang check-clang-tools"'
```
Which meant that Linux ran the clang and clang-tools tests twice. These extra tests were about 24% of the test run and increased testing time (on my local machine) by 45%.

This problem can also happen with other projects but there isn't a simple fix like this one at the moment.
* pstl has a check-pstl target but it is not part of check-all and when I tried it locally I couldn't build it.
* libclc has no check- target.

I will deal with those projects later.

---
Full diff: https://github.com/llvm/llvm-project/pull/111515.diff


1 Files Affected:

- (modified) .ci/generate-buildkite-pipeline-premerge (+1-1) 


``````````diff
diff --git a/.ci/generate-buildkite-pipeline-premerge b/.ci/generate-buildkite-pipeline-premerge
index 98a8b8fff3687a..29e4458610ce66 100755
--- a/.ci/generate-buildkite-pipeline-premerge
+++ b/.ci/generate-buildkite-pipeline-premerge
@@ -198,7 +198,7 @@ function check-targets() {
       echo "check-clang-tools"
     ;;
     compiler-rt)
-      echo "check-all"
+      echo "check-compiler-rt"
     ;;
     cross-project-tests)
       echo "check-cross-project"

``````````

</details>


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


More information about the llvm-commits mailing list