Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Try out using ccache on Travis and build LLVM from source #32839

Closed
wants to merge 1 commit into from

Conversation

alexcrichton
Copy link
Member

We haven't tried this out in awhile, so let's see what the build times look like
if we use ccache.

We haven't tried this out in awhile, so let's see what the build times look like
if we use ccache.
@rust-highfive
Copy link
Collaborator

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member Author

probably want to hold off on merging until we get a good idea on what the travis times look like

@larsbergstrom
Copy link
Contributor

Yeah, and will probably have to get a first full build through to fill the cache and then do a retry to see how much is shaved off.

@brson
Copy link
Contributor

brson commented Apr 8, 2016

r=me

@alexcrichton
Copy link
Member Author

OK looks like an uncached build was 30m longer clocking in a 1h47m, gonna retrigger and see what the cache does now.

@alexcrichton
Copy link
Member Author

Ok this clocked in at 1hr 20m (yay!) so looks like caching does indeed work and shave about a half hour. It's still about 10m slower than our builds which install via apt-get, and I also remember now that we're using that to ensure that we build against the literal 3.7 release, so I don't think we want to do this just yet. If we move more CI to Travis, however, seems like this would indeed be a real win!

@alexcrichton alexcrichton deleted the travis-ccache branch April 11, 2016 15:03
@larsbergstrom
Copy link
Contributor

You could also build in both configs (via the matrix/include stuff, e.g., https://github.com/servo/servo/blob/master/.travis.yml#L3-L5), to ensure that you build against literal 3.7 but check for regressions against LLVM head.

@alexcrichton
Copy link
Member Author

Oh right that's true, forgot to mention that. @brson how do you feel about expanding our Travis matrix? Right now all builds are clocking in at just over an hour, and it looks like we don't have a large backlog, so expanding the matrix shouldn't hopefully flood us too much with a backlog of builds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants