Skip to content
This repository was archived by the owner on Feb 5, 2019. It is now read-only.

Cherry-pick r295964 and closely related part of r282387 #76

Merged
merged 1 commit into from
May 2, 2017
Merged

Cherry-pick r295964 and closely related part of r282387 #76

merged 1 commit into from
May 2, 2017

Conversation

whitequark
Copy link
Member

@TimNN
Copy link

TimNN commented May 1, 2017

Is this fix for rust nightly or beta / stable?

If it is for nightly you'll want to use rust-llvm-2017-04-13 as the base branch, in which case I think that r282387 should already be included.

@whitequark
Copy link
Member Author

If it is for nightly you'll want to use rust-llvm-2017-04-13 as the base branch, in which case I think that r282387 should already be included.

But yesterday's nightly crashes as described in rust-lang/rust#41672.

@TimNN
Copy link

TimNN commented May 1, 2017

But yesterday's nightly crashes as described in rust-lang/rust#41672.

Note that my statement only refers to r282387, not r295964.

@whitequark whitequark changed the base branch from rust-llvm-2016-10-29 to rust-llvm-2017-04-13 May 1, 2017 11:16
@whitequark
Copy link
Member Author

Updated.

@alexcrichton
Copy link
Member

@whitequark would it be possible to do a literal git cherry-pick? It looks like the commit here was crafted by hand?

@whitequark
Copy link
Member Author

@alexcrichton No. There is about a dozen commits between the root branch and the bugfix, all interdependent, and most with significant changes.

@TimNN
Copy link

TimNN commented May 1, 2017

No. There is about a dozen commits between the root branch and the bugfix, all interdependent, and most with significant changes.

@whitequark: Huh? A git cherry-pick of llvm-mirror/llvm@93418ee causes a minor merge conflict in a test but otherwise applies cleanly for me?

@whitequark
Copy link
Member Author

@TimNN Ahhh hm, not sure what went wrong--I must have mixed up something. If you already have the right branch any chance you could simply push it?

@TimNN
Copy link

TimNN commented May 1, 2017

Sure.

The ARMConstantIslandPass didn't have support for handling accesses to
constant island objects through ARM::t2LDRBpci instructions. This adds
support for that.

This fixes PR31997.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@295964 91177308-0d34-0410-b5e6-96231b3b80d8
@TimNN
Copy link

TimNN commented May 1, 2017

I've pushed the updated cherry pick.

@whitequark
Copy link
Member Author

Thanks!

@alexcrichton alexcrichton merged commit 917dcde into rust-lang:rust-llvm-2017-04-13 May 2, 2017
arielb1 pushed a commit that referenced this pull request Jun 27, 2017
Cherry-pick r295964 and closely related part of r282387
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants