[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