This is an archive of the discontinued LLVM Phabricator instance.

[libc++][C++20 modules] Tests bootstrapping build.
AcceptedPublic

Authored by Mordante on Aug 26 2023, 7:34 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Summary

Note that clang-tidy is not enabled in the bootstrap build so only
minimal testing is done for this configuration.

Diff Detail

Event Timeline

Mordante created this revision.Aug 26 2023, 7:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 26 2023, 7:34 AM
Herald added a subscriber: arichardson. · View Herald Transcript
Mordante requested review of this revision.Aug 26 2023, 7:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 26 2023, 7:34 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Sep 6 2023, 8:48 AM
ldionne added a subscriber: ldionne.

LGTM but we should have a green CI run!

libcxx/utils/ci/buildkite-pipeline.yml
351

Commit message: Tests no bootstrap build should probably be Tests bootstrapping build

This revision is now accepted and ready to land.Sep 6 2023, 8:48 AM
Mordante retitled this revision from [libc++][C++20 modules] Tests no bootstrap build. to [libc++][C++20 modules] Tests bootstrap build..Sep 6 2023, 10:49 AM
Mordante retitled this revision from [libc++][C++20 modules] Tests bootstrap build. to [libc++][C++20 modules] Tests bootstrapping build..
Mordante marked an inline comment as done.
Mordante edited the summary of this revision. (Show Details)Oct 31 2023, 12:04 PM
Mordante updated this revision to Diff 557949.Oct 31 2023, 12:32 PM

Rebased to test CI.

Mordante updated this revision to Diff 557950.Oct 31 2023, 12:41 PM

Retrigger CI after removing the abandoned parent patch.