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.

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

Update LLVM submodule #66318

Merged
merged 1 commit into from
Nov 12, 2019
Merged

Update LLVM submodule #66318

merged 1 commit into from
Nov 12, 2019

Conversation

mati865
Copy link
Contributor

@mati865 mati865 commented Nov 11, 2019 •
edited
Loading

Fixes #66315

@rust-highfive
Copy link
Collaborator

r? @cramertj

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

@rust-highfive
Copy link
Collaborator

⚠️ 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 11, 2019
@mati865 mati865 mentioned this pull request Nov 11, 2019
Closed
@matthewjasper
Copy link
Contributor

@bors try

@bors
Copy link
Contributor

bors commented Nov 12, 2019

⌛ Trying commit 63fd531 with merge d283c32...

bors added a commit that referenced this pull request Nov 12, 2019
@bors
Auto merge of #66318 - mati865:llvm-update, r=<try>
d283c32 
[WIP] Llvm update

Asking for a `@bors try`, includes #65608 to verify if it fixes the issue.
@bors
Copy link
Contributor

bors commented Nov 12, 2019

☀️ Try build successful - checks-azure
Build commit: d283c32 (d283c327e8bdabf86af19a76d69a571ec65665bd)

@mati865 mati865 changed the title [WIP] Llvm update Update LLVM submodule Nov 12, 2019
@mati865
Copy link
Contributor Author

mati865 commented Nov 12, 2019

It indeed fixes the issue, updated description so it should be ready to go.

@matthewjasper
Copy link
Contributor

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Nov 12, 2019

📌 Commit 5a1aa8d has been approved by matthewjasper

@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 Nov 12, 2019
@bors
Copy link
Contributor

bors commented Nov 12, 2019

⌛ Testing commit 5a1aa8d with merge 5dda3ee...

bors added a commit that referenced this pull request Nov 12, 2019
@bors
Auto merge of #66318 - mati865:llvm-update, r=matthewjasper
5dda3ee 
Update LLVM submodule

Fixes #66315
@bors
Copy link
Contributor

bors commented Nov 12, 2019

☀️ Test successful - checks-azure
Approved by: matthewjasper
Pushing 5dda3ee to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Nov 12, 2019
@bors bors merged commit 5a1aa8d into rust-lang:master Nov 12, 2019
@rust-highfive
Copy link
Collaborator

📣 Toolstate changed by #66318!

Tested on commit 5dda3ee.
Direct link to PR: #66318

💔 rls on linux: test-pass → test-fail (cc @Xanewok, @rust-lang/infra).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Nov 12, 2019
@rust-highfive
📣 Toolstate changed by rust-lang/rust#66318!
0521e97 
Tested on commit rust-lang/rust@5dda3ee.
Direct link to PR: <rust-lang/rust#66318>

💔 rls on linux: test-pass → test-fail (cc @Xanewok, @rust-lang/infra).
@mati865 mati865 deleted the llvm-update branch November 12, 2019 16:06
@mati865 mati865 mentioned this pull request Nov 13, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cramertj cramertj

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.

LLVM assertion from pointer arithmetic on i686-pc-windows-gnu
5 participants
@mati865 @rust-highfive @matthewjasper @bors @cramertj