Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upRust requires llvm.assume intrinsic and hence LLVM 3.6 #20010
Comments
This comment has been minimized.
This comment has been minimized.
|
FYI, minimal LLVM version was last updated in #5813. The reason was to support inline assembly. |
kmcallister
added
A-LLVM
A-build
labels
Jan 16, 2015
This comment has been minimized.
This comment has been minimized.
|
Just as a note, current rust also fails to build against latest (external) stable LLVM 3.5.1 due to a newer EngineBuilder() signature picked-up by rustllvm/ExecutionEngineWrapper.cpp. |
This comment has been minimized.
This comment has been minimized.
|
Some discussion on this at http://discuss.rust-lang.org/t/targeted-llvm-for-1-0/1371/1. I may give a shot at this if I have some time. |
This comment has been minimized.
This comment has been minimized.
|
Can this be closed now that LLVM 3.5.2 is out? |
This comment has been minimized.
This comment has been minimized.
|
rustc needs at least LLVM 3.6 rc4 (afaik). This issue is regarding the LLVM version check in ./configure, which is still 3.[2-6] (and the error message says "bad LLVM version: ... need >=3.0svn". I believe the ./configure test should be tightened to >=3.6 |
tamird
added a commit
to tamird/rust
that referenced
this issue
May 30, 2015
This comment has been minimized.
This comment has been minimized.
|
configure check was updated to >=3.5 in #23362 and it is still the case today. Rust successfully bootstraps against LLVM 3.5.2 release now. |
anguslees commentedDec 19, 2014
The rust ./configure LLVM version check is only for 3.[2-6], yet #18080 adds a requirement on llvm.assume intrinsic. This intrinsic only appears in LLVM 3.6.
I think the version check needs to be updated accordingly - or only use the new optimisation intrinsic conditionally on building against newer LLVM.