[PATCH] D44880: [Hexagon] peel loops with runtime small trip counts
Max Kazantsev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Mar 25 22:07:09 PDT 2018
mkazantsev added inline comments.
================
Comment at: lib/Transforms/Scalar/LoopUnrollPass.cpp:803
+ // default value 0.
+ computePeelCount(L, LoopSize, UP, TripCount, SE);
if (UP.PeelCount) {
----------------
I don't think it is correct. `computePeelCount` will set PeelCount to 0 if the loop is unpeelable, and you are forcing it to some value without checking this fact.
https://reviews.llvm.org/D44880
More information about the llvm-commits
mailing list