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

Merging r341094: #124

Closed

Conversation

arielb1
Copy link

@arielb1 arielb1 commented Sep 3, 2018


r341094 | efriedma | 2018-08-30 20:59:24 +0200 (Thu, 30 Aug 2018) | 11 lines

[SROA] Fix alignment for uses of PHI nodes.

Splitting an alloca can decrease the alignment of GEPs into the
partition. Normally, rewriting accounts for this, but the code was
missing for uses of PHI nodes and select instructions.

Fixes https://bugs.llvm.org/show_bug.cgi?id=38707 .

Differential Revision: https://reviews.llvm.org/D51335


git-svn-id: https://llvm.org/svn/llvm-project/llvm/branches/release_70@341220 91177308-0d34-0410-b5e6-96231b3b80d8

------------------------------------------------------------------------
r341094 | efriedma | 2018-08-30 20:59:24 +0200 (Thu, 30 Aug 2018) | 11 lines

[SROA] Fix alignment for uses of PHI nodes.

Splitting an alloca can decrease the alignment of GEPs into the
partition.  Normally, rewriting accounts for this, but the code was
missing for uses of PHI nodes and select instructions.

Fixes https://bugs.llvm.org/show_bug.cgi?id=38707 .

Differential Revision: https://reviews.llvm.org/D51335


------------------------------------------------------------------------


git-svn-id: https://llvm.org/svn/llvm-project/llvm/branches/release_70@341220 91177308-0d34-0410-b5e6-96231b3b80d8
@arielb1
Copy link
Author

arielb1 commented Sep 3, 2018

This fixes rust-lang/rust#53181, but is a part of the LLVM 7.0 release branch. Should we instead upgrade to the LLVM 7.0 release branch?

@alexcrichton
Copy link
Member

Thanks! We've moved to rust-llvm-release-8-0-0-v1 in rust-lang/rust#53611 so rust-llvm-release-7-0-0-v2 is no longer used on master. I'd be fine either backporting this commit to that or just moving forward the LLVM submodule, unfortunately that branch was branched late enough that it doesn't include this commit :(

@arielb1 arielb1 closed this Sep 6, 2018
@arielb1 arielb1 reopened this Sep 6, 2018
@arielb1 arielb1 closed this Sep 6, 2018
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.

3 participants