[PATCH] D37640: [GISel]: Fix generation of illegal COPYs (of different sizes) during CallLowering

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 6 11:06:17 PDT 2017


rovka added a comment.

Hi Aditya,

I uploaded the ARM changes here: https://reviews.llvm.org/differential/diff/118037/

As a sidenote, this caused trouble with some of my integration tests because now the legalize combiner finds all sorts of opportunities to replace trunc + ext with shifts, which weren't implemented on ARM yet (they are now). I don't think the combiner should be introducing instructions without first checking if they are legal, maybe you can consider fixing that in the future.


https://reviews.llvm.org/D37640





More information about the llvm-commits mailing list