This is an archive of the discontinued LLVM Phabricator instance.

[test-release] Build BOLT by default for x86/arm
ClosedPublic

Authored by rafauler on Nov 2 2022, 6:10 PM.

Details

Summary

Make BOLT build by default in X86 and AArch64 Linux boxes.

Diff Detail

Event Timeline

rafauler created this revision.Nov 2 2022, 6:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 2 2022, 6:10 PM
rafauler requested review of this revision.Nov 2 2022, 6:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 2 2022, 6:10 PM
thieta accepted this revision.Nov 3 2022, 1:29 AM

looks fine with a small comment.

llvm/utils/release/test-release.sh
54

why not x86_64 | arm64 | aarch64 instead of two lines?

This revision is now accepted and ready to land.Nov 3 2022, 1:29 AM
xbolva00 accepted this revision.Nov 3 2022, 2:53 AM

Great, thanks

xbolva00 added inline comments.Nov 3 2022, 3:51 AM
llvm/utils/release/test-release.sh
14–15

Move Machine here?

rafauler updated this revision to Diff 473012.Nov 3 2022, 1:19 PM

Implement suggestions

This revision was landed with ongoing or failed builds.Nov 3 2022, 1:25 PM
This revision was automatically updated to reflect the committed changes.
bcain added a subscriber: bcain.Nov 9 2022, 4:40 PM