[PATCH] D19381: Extend load/store type canonicalization to handle unordered operations

[JF Bastien via llvm-commits]

jfb added a comment.

In http://reviews.llvm.org/D19381#409535, @reames wrote:

> This landed as 267210.
>
> @JF - There shouldn't be any code generation differences for two types with the same alignment, even if that alignment isn't natural.


Many ISAs have different load / store instructions for FP versus integer, and some behave differently. Going from FP to integer in this case (no intervening use) seems fine, but I do think it's likely to change code generation (unless the backend already performed the same transformation).


http://reviews.llvm.org/D19381