[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