Skip to content

Update LLVM submodule #66882

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 7, 2019
Merged

Update LLVM submodule #66882

merged 1 commit into from
Dec 7, 2019

Conversation

mati865
Copy link
Contributor

@mati865 mati865 commented Nov 29, 2019 •
edited
Loading

Fixes #63959

Credit for finding the cause goes to eddyb.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@rust-highfive
Copy link
Contributor

⚠️ Warning ⚠️

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 29, 2019
@mati865 mati865 force-pushed the znver1_fix branch 2 times, most recently from 075a6e7 to 91348f0 Compare November 30, 2019 00:30
@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 2, 2019

📌 Commit 049ca74 has been approved by nikomatsakis

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 2, 2019
@bors
Copy link
Collaborator

bors commented Dec 6, 2019

⌛ Testing commit 049ca74 with merge c9d4070e9b2dac07b421610e1c486a4e143d2397...

@rust-highfive
Copy link
Contributor

The job i686-apple of your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@bors
Copy link
Collaborator

bors commented Dec 6, 2019

💔 Test failed - checks-azure

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Dec 6, 2019
@mati865
Copy link
Contributor Author

mati865 commented Dec 6, 2019

Spurious failure:

failed to get 200 response from https://crates.io/api/v1/crates/rand_chacha/0.1.0/download, got 503

@mati865
Copy link
Contributor Author

mati865 commented Dec 6, 2019

@nikomatsakis could you retry it?
Thanks in advance.

@nikic
Copy link
Contributor

nikic commented Dec 6, 2019

@bors retry

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 6, 2019
@bors
Copy link
Collaborator

bors commented Dec 7, 2019

⌛ Testing commit 049ca74 with merge 0a953cd...

bors added a commit that referenced this pull request Dec 7, 2019
@bors
Auto merge of #66882 - mati865:znver1_fix, r=nikomatsakis
0a953cd 
Update LLVM submodule

Fixes #63959

Credit for finding the cause goes to `eddyb`.
@bors
Copy link
Collaborator

bors commented Dec 7, 2019

☀️ Test successful - checks-azure
Approved by: nikomatsakis
Pushing 0a953cd to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Dec 7, 2019
@bors bors merged commit 049ca74 into rust-lang:master Dec 7, 2019
@mati865 mati865 deleted the znver1_fix branch December 7, 2019 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nikomatsakis nikomatsakis

Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Miscompilation with target-cpu=znver1 (AMD Ryzen 1000/2000 series) on Windows + LLVM 9.
5 participants
@mati865 @rust-highfive @nikomatsakis @bors @nikic