-
-
Notifications
You must be signed in to change notification settings - Fork 261
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
Fix LLVM 5.0 build #2173
Fix LLVM 5.0 build #2173
Conversation
I'll have a new LLVM up for the SPIRV OSX box soon. |
Done. Not sure how to force an de-chaching. |
You could version the llvm-spirv dir ( |
Yep, just version the directory name. |
Thanks Johan. Sorry I working like mad on my honours, I'll be out of action until 5th of July. |
SPIRV build is green again. |
driver/main.cpp
Outdated
// This function exits the program. | ||
void printVersion(llvm::raw_ostream &OS) { | ||
OS << "LDC - the LLVM D compiler (" << global.ldc_version << "):\n"; | ||
OS << " based on DMD " << global.version << "and LLVM " << global.llvm_version << "\n"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space before and LLVM
.
:( travis jobs keep running out of time |
Yeah on weekends the Travis performance is a catastrophe. :/ |
It looks like we really need to find an alternative to Travis, or speed up our builds by a lot :( |
Not that it would help with the timeouts on Travis, but I am happy to setup auto-merge with the Dlang-Bot in case that would help you. Currently it doesn't restart failing transient Travis builds yet, but that is on our TODO list (of course it will happen faster with a PR). |
Rebased after merging the Travis & dmd-testsuite tweaks in #2074. Travis time-outs should now be exceedingly rare. Circle a) relies on the flaky LLVM APT repo and b) currently exhibits a new/temporary (?) failure of |
Circle has had that GDB failure for quite some time (months), unless it got fixed and its a new failure. |
* Fix LLVM 5.0 compilation. * Fix adding linker option metadata to LLVM module. See https://reviews.llvm.org/D31349 * Version the SPIRV cache dir on travis.
No description provided.