[PATCH] D114453: [unroll] Split full exact and full bound unroll costing [NFC]

[Philip Reames via Phabricator via llvm-commits]

reames created this revision.
reames added reviewers: lebedev.ri, fhahn, anna, Ali-Sed, nikic.
Herald added subscribers: zzheng, bollu, hiraditya, mcrosier.
reames requested review of this revision.
Herald added a project: LLVM.

This change should be NFC.  It's posted for review mostly to make sure others are happy with the names I'm introducing for "exact full unroll" and "bounded full unroll".  The motivation here is that our cost model for bounded unrolling is too aggressive - it gives benefits for exits we aren't going to prune - but I also just think the new version of the code is a lot easier to follow.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D114453

Files:
  llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D114453.389237.patch
Type: text/x-patch
Size: 4568 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211123/53169480/attachment.bin>