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

[Ulrich Weigand via Phabricator via llvm-commits]

uweigand added a comment.

I guess this is OK with me, but I admit I don't quite understand why this would make any difference ...  As far as I can see, the flavor of DAG.getLoad currently used would use the existing MemOperand, including its existing alignment, while the flavor in your patch creates a new MemOperand using the passed alignment value which the patch also gets from the current MemOperand.  Either way we ought to end up with the same alignment ...

Would you mind explaining what I'm missing here?


https://reviews.llvm.org/D31609