Skip to content

Travis: use LLVM3.9 on OS X#1817

Merged
JohanEngelen merged 1 commit intoldc-developers:masterfrom
JohanEngelen:mac39
Nov 25, 2016
Merged

Travis: use LLVM3.9 on OS X#1817
JohanEngelen merged 1 commit intoldc-developers:masterfrom
JohanEngelen:mac39

Conversation

@JohanEngelen
Copy link
Member

Let's see if this works (so that we can release with LLVM3.9 on OS X).

@JohanEngelen
Copy link
Member Author

@JohanEngelen
Copy link
Member Author

Instead of waiting for homebrew, I rewrote travis.yml to download the binaries from LLVM directly and cache them like we do for the Linux builds. This appears to speed up the build a lot (~39min -> ~30min), possibly because LLVM's binaries have all assertions disabled?

@JohanEngelen
Copy link
Member Author

(green for mac with LLVM3.9 btw, nice :-))

@kinke
Copy link
Member

kinke commented Nov 9, 2016

possibly because LLVM's binaries have all assertions disabled

Seems like it, as we don't have assertions enabled on Linux as well. I must say I really prefer CI builds with enabled assertions; people keep complaining about LDC crashing, and most of these issues (and others!) would be caught by enabled assertions. I would even prefer alphas and betas to ship with assertions too.
Does CircleCI have assertions enabled or is it just AppVeyor now?

@JohanEngelen
Copy link
Member Author

Pretty sure that CircleCI also does not have assertions enabled...

Hm.

@JohanEngelen JohanEngelen mentioned this pull request Nov 18, 2016
@JohanEngelen
Copy link
Member Author

Already merged in release-1.1.0 branch.

@JohanEngelen JohanEngelen merged commit 13a97ad into ldc-developers:master Nov 25, 2016
@JohanEngelen JohanEngelen deleted the mac39 branch November 25, 2016 09:58
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.

2 participants