Skip to content

Conversation

@dotdash
Copy link
Contributor

@dotdash dotdash commented Sep 11, 2015

No description provided.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@dotdash dotdash force-pushed the llvm_fix branch 2 times, most recently from 17a079d to d30a0b7 Compare September 11, 2015 15:37
@eddyb
Copy link
Member

eddyb commented Sep 11, 2015

@bors r+

@bors
Copy link
Collaborator

bors commented Sep 11, 2015

📌 Commit e8c398d has been approved by eddyb

@alexcrichton
Copy link
Member

Out of curiosity, what were these build failures that were seen? Is this something we should be concerned about when using stock LLVM 3.7?

@alexcrichton
Copy link
Member

Also, just FYI, but I think that future LLVM updates can drop rust-lang/llvm@3c88bf7 as the upstream bug was fixed in 3.8.

@alexcrichton
Copy link
Member

@bors: p=1

Looks like this is why we haven't produced a nightly recently

@bors
Copy link
Collaborator

bors commented Sep 11, 2015

⌛ Testing commit e8c398d with merge 33f0920...

bors added a commit that referenced this pull request Sep 11, 2015
@bors bors merged commit e8c398d into rust-lang:master Sep 12, 2015
@dotdash dotdash deleted the llvm_fix branch January 15, 2016 08:40
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.

6 participants