This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Add support for bootstrapping build in libcxx-lit
ClosedPublic

Authored by hawkinsw on Aug 3 2023, 3:35 PM.

Details

Summary

Add a [-b|--bootstrap] flag to change cxx-test-depends to
runtimes-test-depends when testing a build of libcxx from bootstrapping.

Signed-off-by: Will Hawkins <hawkinsw@obs.cr>

Diff Detail

Event Timeline

hawkinsw created this revision.Aug 3 2023, 3:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 3 2023, 3:35 PM
Herald added a subscriber: arichardson. · View Herald Transcript
hawkinsw requested review of this revision.Aug 3 2023, 3:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 3 2023, 3:35 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript

Hello everyone! It took me a bit to figure out that cxx-test-depends needed to be runtimes-test-depends and where/how to change that when using the libcxx-lit and thought that this little change might help!

hawkinsw updated this revision to Diff 547028.Aug 3 2023, 3:40 PM

Incorrectly marked the patch as libc and not libc++.

hawkinsw retitled this revision from [libc] Add support for runtimes build in libcxx-lit to [libc++] Add support for runtimes build in libcxx-lit.Aug 4 2023, 8:15 AM
hawkinsw added a reviewer: philnik.

@philnik Sorry to bother by adding you as a reviewer. Based on this I thought you might have some input on this little tweak! Thank you for all that you are doing!
Will

Since we're trying to move away from "runtimes" build and towards using the terminology "bootstapping" build, can we reflect that in the option name? Otherwise this looks like a good idea to me.

Since we're trying to move away from "runtimes" build and towards using the terminology "bootstapping" build, can we reflect that in the option name? Otherwise this looks like a good idea to me.

Absolutely! Thanks for the review!

hawkinsw updated this revision to Diff 547917.Aug 7 2023, 12:41 PM

Updating to better use bootstrap instead of runtimes based on @ldionne's feedback!

hawkinsw retitled this revision from [libc++] Add support for runtimes build in libcxx-lit to [libc++] Add support for bootstrapping build in libcxx-lit.Aug 7 2023, 12:47 PM
hawkinsw edited the summary of this revision. (Show Details)
ldionne accepted this revision.Sep 5 2023, 2:37 PM

LGTM. CI issues are unrelated.

This revision is now accepted and ready to land.Sep 5 2023, 2:37 PM
This revision was landed with ongoing or failed builds.Sep 7 2023, 9:05 AM
This revision was automatically updated to reflect the committed changes.