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 submodule #66318

Merged
merged 1 commit into from Nov 12, 2019

Conversation

@mati865
Copy link
Contributor

mati865 commented Nov 11, 2019

Fixes #66315

@rust-highfive

This comment has been minimized.

Copy link
Collaborator

rust-highfive commented Nov 11, 2019

r? @cramertj

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

@rust-highfive

This comment has been minimized.

Copy link
Collaborator

rust-highfive commented Nov 11, 2019

⚠️ Warning ⚠️

  • These commits modify submodules.
@matthewjasper

This comment has been minimized.

Copy link
Contributor

matthewjasper commented Nov 12, 2019

@bors try

@bors

This comment has been minimized.

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
[WIP] Llvm update

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

This comment has been minimized.

Copy link
Contributor

bors commented Nov 12, 2019

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

@mati865 mati865 force-pushed the mati865:llvm-update branch from 63fd531 to 5a1aa8d Nov 12, 2019
@mati865 mati865 changed the title [WIP] Llvm update Update LLVM submodule Nov 12, 2019
@mati865

This comment has been minimized.

Copy link
Contributor Author

mati865 commented Nov 12, 2019

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

@matthewjasper

This comment has been minimized.

Copy link
Contributor

matthewjasper commented Nov 12, 2019

@bors r+ p=1

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Nov 12, 2019

📌 Commit 5a1aa8d has been approved by matthewjasper

@bors

This comment has been minimized.

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
Update LLVM submodule

Fixes #66315
@bors

This comment has been minimized.

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 label Nov 12, 2019
@bors bors merged commit 5a1aa8d into rust-lang:master Nov 12, 2019
5 checks passed
5 checks passed
homu Test successful
Details
pr Build #20191112.24 succeeded
Details
pr (Linux mingw-check) Linux mingw-check succeeded
Details
pr (Linux x86_64-gnu-llvm-6.0) Linux x86_64-gnu-llvm-6.0 succeeded
Details
pr (Linux x86_64-gnu-tools) Linux x86_64-gnu-tools succeeded
Details
@rust-highfive

This comment has been minimized.

Copy link
Collaborator

rust-highfive commented Nov 12, 2019

📣 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
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 mati865:llvm-update branch Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.