Skip to content

Update LLVM submodule #52952

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Update LLVM submodule #52952

wants to merge 1 commit into from

Conversation

cramertj
Copy link
Member

@cramertj cramertj commented Aug 1, 2018

@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 Aug 1, 2018
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Aug 1, 2018

📌 Commit 01cf9bf has been approved by alexcrichton

@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 Aug 1, 2018
@alexcrichton alexcrichton mentioned this pull request Aug 2, 2018
Merged
cramertj added a commit to cramertj/rust that referenced this pull request Aug 2, 2018
@cramertj
Rollup merge of rust-lang#52952 - cramertj:llvm-update, r=alexcrichton
4889e0f 
Update LLVM submodule

Fix rust-lang#52884

r? @alexcrichton
@cramertj cramertj mentioned this pull request Aug 2, 2018
Closed
cramertj added a commit to cramertj/rust that referenced this pull request Aug 2, 2018
@cramertj
Rollup merge of rust-lang#52952 - cramertj:llvm-update, r=alexcrichton
1727484 
Update LLVM submodule

Fix rust-lang#52884

r? @alexcrichton
@cramertj cramertj mentioned this pull request Aug 2, 2018
Closed
@kennytm
Copy link
Member

kennytm commented Aug 3, 2018 •
edited
Loading

@bors r-

Obsoleted by #52983 (comment) (rust-lang/llvm@59a658d specifically), so removing this from queue to avoid merge conflict.

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 3, 2018
@kennytm kennytm added S-blocked Status: Blocked on something else such as an RFC or other implementation work. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 3, 2018
@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-blocked Status: Blocked on something else such as an RFC or other implementation work. labels Aug 3, 2018
@kennytm kennytm added S-blocked Status: Blocked on something else such as an RFC or other implementation work. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 3, 2018
@alexcrichton
Copy link
Member

Note that #52983 is also much higher risk, so if it bounces once we should put this back in the queue

@kennytm
Copy link
Member

kennytm commented Aug 6, 2018

#52983 has been merged, closing this one.

@kennytm kennytm closed this Aug 6, 2018
@cramertj cramertj deleted the llvm-update branch August 6, 2018 16:49
This was referenced Aug 31, 2018
Closed
Merged
bors added a commit that referenced this pull request Sep 4, 2018
@bors
Auto merge of #53939 - parched:beta-aarch64-fix, r=alexcrichton
82626e6 
Update LLVM to fix segfault on AArch64

Fixes #53742

This is a backport of #52952 to beta

cc @alexcrichton
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@alexcrichton alexcrichton

Labels
S-blocked Status: Blocked on something else such as an RFC or other implementation work.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@cramertj @rust-highfive @alexcrichton @bors @kennytm