[PATCH] D30757: [LoopUnroll] Don't peel loops where the latch isn't the exiting block

Michael Zolotukhin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 16 13:54:56 PDT 2017


mzolotukhin added a comment.

This version looks good to me too.

MichaelZ

PS: Can we have a similar problem in the regular loop unrolling?



================
Comment at: test/Transforms/LoopUnroll/peel-loop-irreducible.ll:36-37
+
+exit:
+  ret i32 %sum
+}
----------------
I'd prefer to have LCSSA-ed version of the loop in the test. And, by the way, do we need `%sum`, `%plus` and `%incsum` at all? Maybe remove them to make the test even smaller?


https://reviews.llvm.org/D30757





More information about the llvm-commits mailing list