[PATCH] D18028: DAGCombiner: Relax alignment restriction when changing store type

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 13:17:50 PDT 2016


t.p.northover added a comment.

I think the ARM change is probably fine. There's real ARM code designed to transform a VMOVDRR/VSTR into two STRs (for legitimate, if debatable reasons: vmov was really slow on some cores), it just doesn't happen to trigger if the type is a vector.

Tim.


http://reviews.llvm.org/D18028





More information about the llvm-commits mailing list