[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