[PATCH] D44919: [LoopUnroll][NFC] Remove redundant canPeel check
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 27 01:30:22 PDT 2018
fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.
LGTM. I think it is very reasonable to expect the caller to only pass in a peel count > 0 if we can acutally peel the loop. There is no need to check twice. Also, currently computePeelCount overrides the target specific settings, so I don't think this change would effect any downstream backends forcing a peel count.
https://reviews.llvm.org/D44919
More information about the llvm-commits
mailing list