[PATCH] D31609: [SystemZ] Fix bitcast of load translation to preserve alignment.

Nirav Dave via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 5 06:30:47 PDT 2017


niravd added a comment.

In https://reviews.llvm.org/D31609#718978, @uweigand wrote:

> Ah, I think the problem is simply that this piece of code in lowerBITCAST does not expect to be handling extending loads at all, and does the wrong thing for them.  It probably just should verify this, and fall through to the rest of the code if the load is actually an extending one ...


This seems reasonable. We can change line 2794 to check isNormalLoad(N) and from my quick look at SystemZISelLowering it looks like Extended Loads have no direct realization.


https://reviews.llvm.org/D31609





More information about the llvm-commits mailing list