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

Update LLVM to rust-llvm-2015-02-19 #22526

Merged
merged 1 commit into from
Feb 24, 2015
Merged

Update LLVM to rust-llvm-2015-02-19 #22526

merged 1 commit into from
Feb 24, 2015

Conversation

dotdash
Copy link
Contributor

@dotdash dotdash commented Feb 19, 2015

Fixes #22159
Fixes #21721

@rust-highfive
Copy link
Contributor

r? @huonw

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

@alexcrichton
Copy link
Member

@bors: r+ d6d405e

@Manishearth
Copy link
Member

There seems to be a merge conflict, and this might be the cause of a failure on android

http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3681/steps/test/logs/stdio

error: undefined reference to 'pthread_atfork'

@Manishearth
Copy link
Member

Not your fault. You are back in the rollup 😄

Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 20, 2015
@Manishearth
Copy link
Member

@bors: r=alexcrichton 6957bb6 p=1

@bors
Copy link
Collaborator

bors commented Feb 21, 2015

⌛ Testing commit 6957bb6 with merge 1ce1596...

bors added a commit that referenced this pull request Feb 21, 2015
@Manishearth
Copy link
Member

(let's let this one merge)

@bors
Copy link
Collaborator

bors commented Feb 21, 2015

💔 Test failed - auto-win-64-nopt-t

@dotdash
Copy link
Contributor Author

dotdash commented Feb 21, 2015

@bors retry

@bors
Copy link
Collaborator

bors commented Feb 21, 2015

⌛ Testing commit 6957bb6 with merge d8bf952...

@bors
Copy link
Collaborator

bors commented Feb 21, 2015

💔 Test failed - auto-win-64-opt

@Manishearth
Copy link
Member

@bors: retry

@bors
Copy link
Collaborator

bors commented Feb 22, 2015

⌛ Testing commit 6957bb6 with merge fad7011...

@bors
Copy link
Collaborator

bors commented Feb 22, 2015

💔 Test failed - auto-linux-32-nopt-t

@Manishearth
Copy link
Member

@bors: retry

@Manishearth
Copy link
Member

@bors: p=10

@bors
Copy link
Collaborator

bors commented Feb 22, 2015

⌛ Testing commit 6957bb6 with merge 7394bd2...

bors added a commit that referenced this pull request Feb 22, 2015
@bors
Copy link
Collaborator

bors commented Feb 22, 2015

💔 Test failed - auto-win-32-opt

@Manishearth
Copy link
Member

@bors: retry

@bors
Copy link
Collaborator

bors commented Feb 22, 2015

⌛ Testing commit 6957bb6 with merge 55704ca...

@bors
Copy link
Collaborator

bors commented Feb 22, 2015

💔 Test failed - auto-mac-32-opt

@Manishearth
Copy link
Member

This pull request has incredibly bad luck 😭

@Manishearth
Copy link
Member

@bors: retry

@bors
Copy link
Collaborator

bors commented Feb 23, 2015

⌛ Testing commit 6957bb6 with merge 4566027...

@Manishearth
Copy link
Member

Oh, great, all-seeing buildbot in the sky, whatever we have done to offend you, please forgive us this time and let this pull request may land. Thank you ❤️

@bors
Copy link
Collaborator

bors commented Feb 23, 2015

💔 Test failed - auto-mac-64-nopt-t

@Manishearth
Copy link
Member

Okay, that's new (for this PR)


command timed out: 1200 seconds without output, attempting to kill
process killed by signal 13
program finished with exit code -1
elapsedTime=2752.439075

whilst testing sync/threads. Possible deadlock? I think I've seen this before as an intermittent, but could someone verify on a mac if this failure is reproducible?

I'm not sure, but I recall trying out stdtest on the rollup when it contained this and it passed (on Linux)

I'm spawning another build to test this locally on Linux again.

@alexcrichton
Copy link
Member

@bors: retry

That's just stdtest deadlocking and happens somewhat regularly unfortunately on all platforms.

@bors
Copy link
Collaborator

bors commented Feb 23, 2015

⌛ Testing commit 6957bb6 with merge a389003...

@Manishearth
Copy link
Member

@alexcrichton Yeah, but with this PR I'm starting to suspect that something might be somehow broken (given the number of failures it's endured)

@bors
Copy link
Collaborator

bors commented Feb 23, 2015

💔 Test failed - auto-win-32-nopt-t

@alexcrichton
Copy link
Member

@bors: retry

1 similar comment
@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Collaborator

bors commented Feb 23, 2015

⌛ Testing commit 6957bb6 with merge 4b779ec...

@bors
Copy link
Collaborator

bors commented Feb 23, 2015

💔 Test failed - auto-win-32-nopt-t

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Collaborator

bors commented Feb 23, 2015

⌛ Testing commit 6957bb6 with merge c4fe7d6...

bors added a commit that referenced this pull request Feb 23, 2015
@bors bors merged commit 6957bb6 into rust-lang:master Feb 24, 2015
@dotdash dotdash deleted the llvmup branch March 1, 2015 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants