aaron.ballman added a comment.

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++?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155078

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to