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

Move Travis onto llvm-3.3 #3900

Merged
merged 2 commits into from
Aug 1, 2013
Merged

Conversation

staticfloat
Copy link
Member

This updates Travis to use LLVM 3.3 instead of 3.2, and simultaneously heralds the arrival of 3.3 binaries on the PPA. 3.2 binaries are maintained alongside, so users can switch at their leisure, for now.

@JeffBezanson; do you want to move to 3.3 for the 0.2 release? I can't find a reason not to, we have a PPA, Homebrew has a formula, and even Arch Linux has a package for it, so I don't think we're going to lack in distribution support. If so, we can merge this, I'll change my nightlies to require 3.3 instead of 3.2, and we can change the requisite setting in deps/Version.make. We just have to be careful to clear out anything having to do with LLVM in usr/ after the version is changed in deps/Version.make.

@ViralBShah
Copy link
Member

It would be nice to move to llvm 3.3. I know it is late in the release process, but we still have another 2-3 weeks of the release period ahead of us, and the freeze is not yet on.

I would be more conservative once we have more releases and a larger user base. This also fixes some open issues related to LLVM, which makes it more desirable.

@JeffBezanson
Copy link
Member

Yes, I'm in favor of this since it fixes a specific bug.

@ViralBShah
Copy link
Member

Can you go ahead and update the LLVM_VER?

@JeffBezanson
Copy link
Member

Hey uh, @loladiro @vtjnash do you remember which bug(s) llvm 3.3 fixes for us?

@Keno
Copy link
Member

Keno commented Aug 1, 2013

AVX feature detection (#3634)

@JeffBezanson
Copy link
Member

Ah, the issue was already closed, which is why I wasn't finding it.

@staticfloat
Copy link
Member Author

Note this also affected select EC2 users.

@Keno
Copy link
Member

Keno commented Aug 1, 2013

Only the ones in Ireland though (Remember the "Julia crashes in Ireland" issue? - Same problem but LLVM not OpenBLAS).

JeffBezanson added a commit that referenced this pull request Aug 1, 2013
@JeffBezanson JeffBezanson merged commit 3745784 into JuliaLang:master Aug 1, 2013
@staticfloat staticfloat deleted the llvm33 branch July 30, 2014 17:51
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