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

rustc: Fix support for LLVM 3.3 #12719

Merged
merged 1 commit into from
Mar 6, 2014
Merged

Conversation

alexcrichton
Copy link
Member

The llvm.copysign and llvm.round intrinsics weren't added until LLVM 3.4, so if
we're on LLVM 3.3 we lower these to calls in libm instead of LLVM intrinsics.

This should fix our travis failures.

@thestinger
Copy link
Contributor

We could install LLVM 3.4 from a PPA in travis as rust-ci does with the third party Rust nightlies. LLVM actually has official repositories for Ubuntu: http://llvm.org/apt/

@alexcrichton
Copy link
Member Author

I may not be doing it correctly, but I am unable to find the 3.4 PPAs

https://travis-ci.org/alexcrichton/rust/builds/20164061

The llvm.copysign and llvm.round intrinsics weren't added until LLVM 3.4, so if
we're on LLVM 3.3 we lower these to calls in libm instead of LLVM intrinsics.

This should fix our travis failures.
@thestinger
Copy link
Contributor

@alexcrichton: I think it needs llvm-toolchain-precise-3.4 instead of llvm-toolchain-precise

@alexcrichton
Copy link
Member Author

Indeed it does! I'm experimenting with something for travis which I hope to put up soon.

bors added a commit that referenced this pull request Mar 6, 2014
The llvm.copysign and llvm.round intrinsics weren't added until LLVM 3.4, so if
we're on LLVM 3.3 we lower these to calls in libm instead of LLVM intrinsics.

This should fix our travis failures.
@bors bors closed this Mar 6, 2014
@bors bors merged commit 9396452 into rust-lang:master Mar 6, 2014
@alexcrichton alexcrichton deleted the fix-llvm-33 branch March 6, 2014 20:01
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
… r=jonas-schievink

Remove unnecessary unsafe from format_args expansion
flip1995 pushed a commit to flip1995/rust that referenced this pull request May 17, 2024
…18, r=xFrednet

Fix website dash replacement

Fixes: rust-lang#12718
changelog: Made clippy lints website search replace all occurrences of dashes with underscores instead of only the first one. This is in order to allow the user to search for lints with more than two words using dahses.
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