Differential D26747

Speed up Cirrus CI by using CROSS_TOOLCHAIN
ClosedPublic
Actions

Authored by arichardson on Oct 12 2020, 10:31 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Sep 27, 11:06 AM

	Unknown Object (File)
	Mon, Sep 23, 5:06 AM

	Unknown Object (File)
	Mon, Sep 23, 3:58 AM

	Unknown Object (File)
	Sat, Sep 21, 9:55 AM

	Unknown Object (File)
	Mon, Sep 16, 4:09 PM

	Unknown Object (File)
	Sep 6 2024, 8:55 PM

	Unknown Object (File)
	Sep 1 2024, 5:05 PM

	Unknown Object (File)
	Aug 9 2024, 11:15 AM

View All 32 Files

Subscribers

Details

Reviewers

Commits

rS367102: Speed up Cirrus CI by using CROSS_TOOLCHAIN

Summary

Installing the llvm11 package instead of bootstrapping it from the source
tree reduces the build time by about 20 minutes.

The last freebsd/freebsd build that was tested (r366629) took 1h 21m 22s,
whereas my GitHub fork with this .cirrus.yml took 58m 6s.
We could probably further reduce time by using images that have LLVM
pre-installed: the pkg install step took 4 minutes 30s.

Since the bootstrap toolchain is still tested by Jenkins, this should not
reduce test coverage of the CI script.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 34108
Build 31277: arc lint + arc unit

Event Timeline

arichardson requested review of this revision.Oct 12 2020, 10:31 AM

arichardson created this revision.

Harbormaster completed remote builds in B34108: Diff 78124.Oct 12 2020, 10:31 AM

ping?

emaste accepted this revision.Oct 27 2020, 12:46 PM

emaste added inline comments.

.cirrus.yml
4–5	In a few days we'll have a 12.2 image available. I'm happy for this to go in now and switch it later.

This revision is now accepted and ready to land.Oct 27 2020, 12:46 PM

Closed by commit rS367102: Speed up Cirrus CI by using CROSS_TOOLCHAIN (authored by arichardson). · Explain WhyOct 28 2020, 11:54 AM

This revision was automatically updated to reflect the committed changes.

arichardson added a commit: rS367102: Speed up Cirrus CI by using CROSS_TOOLCHAIN.

Herald added a subscriber: imp. · View Herald TranscriptOct 28 2020, 11:54 AM

Revision Contents
Changeset List

Path

Size

10 lines

Diff 78124

.cirrus.yml

Loading...