[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