Skip to content

Travis: Add LLVM 5.0#2316

Merged
kinke merged 1 commit intoldc-developers:masterfrom
JohanEngelen:travis50
Sep 10, 2017
Merged

Travis: Add LLVM 5.0#2316
kinke merged 1 commit intoldc-developers:masterfrom
JohanEngelen:travis50

Conversation

@JohanEngelen
Copy link
Member

LLVM 5.0 was just released.

@kinke
Copy link
Member

kinke commented Sep 8, 2017

There's still no prebuilt archive for Debian x64 (from LLVM directly), they gotta be kidding.

@JohanEngelen
Copy link
Member Author

Ubuntu x64 is there, just the filename is following a different pattern :/

@JohanEngelen
Copy link
Member Author

Wow, we have a conflicting cmdline option "dwarf-version" with LLVM 5.0, and it doesn't bug on my machine. :(
(I really think we should put all LLVM options behind -llvm like clang does, so we won't have these collisions and can remove the manual list of options to be removed from the help)

@JohanEngelen
Copy link
Member Author

Perhaps the LLVM binaries are a strange build. The -dwarf-version option is not supposed to be in the LLVM libs that we link with...?

@kinke
Copy link
Member

kinke commented Sep 8, 2017

You're most likely building with LLD, right? LLD 5.0+ needs #2148.

@JohanEngelen
Copy link
Member Author

Ah, yes it's building with LLD.

@JohanEngelen
Copy link
Member Author

I've added -DLDC_WITH_LLD=OFF in a spot where it's easily spotted, reminding us to enable it when it works.

@kinke kinke merged commit 6ad0193 into ldc-developers:master Sep 10, 2017
@JohanEngelen JohanEngelen deleted the travis50 branch September 11, 2017 16:34
kinke pushed a commit to kinke/ldc that referenced this pull request Sep 12, 2017
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