[PATCH] D21620: [LV] Don't widen trivial induction variables
Matthew Simpson via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 1 11:57:06 PDT 2016
mssimpso added a comment.
Thanks for the reviews Adam. http://reviews.llvm.org/D21903 is indeed the prequel refactoring patch, and I've rebased the current patch on top of that one. Lines 2250-2257 in the current patch are the primary functional change now. This check is now only in one place (in the new widenIntInduction function I added), unlike before the refactoring, where it was in two places (with the int induction case and the trunc case). Hope that helps!
http://reviews.llvm.org/D21620
More information about the llvm-commits
mailing list