[libcxx-commits] [PATCH] D155078: [ci] Make libc++ and Clang CI scripts independent

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

In D155078#4493995 <https://reviews.llvm.org/D155078#4493995>, @aaron.ballman wrote:

> In D155078#4493468 <https://reviews.llvm.org/D155078#4493468>, @ldionne wrote:
>
>> Ok, this is ready to go. More context around this change:
>>
>> I created a clang-specific CI pipeline (https://buildkite.com/llvm-project/clang-ci) and Clang CI jobs will now trigger inside that pipeline instead of inside the libc++ pipeline (https://buildkite.com/llvm-project/libcxx-ci). The problem this solves is that 90% of the jobs of the libc++ CI pipelines are otherwise Clang jobs, which makes it nearly impossible for libc++ to figure out information about the jobs we're actually running.
>>
>> - For now, this does not have any impact on the fact that Clang CI is still utilizing libc++ resources (although this is next on my list of things to address).
>> - This doesn't have any impact on whether we'll eventually want to move to Github Actions or some other CI system. This simply fixes the current system before the LLVM 17 release. In particular, I would like to avoid bikeshedding the location of the `generate-buildkite-pipeline` scripts under `.ci` -- it doesn't matter, I just wanted a neutral (non-libcxx-specific) location and that can be changed later once CI has stabilized.
>
> Please excuse my ignorance, but does this change precommit CI test coverage when reviews are created/updated? e.g., if a review goes up with changes only to Clang, will the review still get a libc++ test run to ensure the Clang changes don't break libc++?

It actually fixes it. Previously, if both libc++ and Clang were modified in a review, only the libc++ CI was triggered. After this patch:

- If Clang is modified, the Clang pre-commit CI is triggered (which includes running the libc++ test suite w/ just-built clang in C++03, C++20 and Modules IIRC)
- If Libc++ is modified, the libc++ pre-commit CI is triggered (which includes the 60ish jobs we have)
- If both are modified, both are triggered.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155078/new/

https://reviews.llvm.org/D155078