[PATCH] D106001: [WIP][NFC] factor out unrolling decision logic

Ali Sedaghati via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 27 11:40:06 PDT 2021


Ali-Sed updated this revision to Diff 362120.
Ali-Sed marked 2 inline comments as done.
Ali-Sed added a comment.

[NFC] Decoupling the unrolling logic into three different functions. The shouldPragmaUnroll() covers the 1st and 2nd priorities of the previous code, the shouldFullUnroll() covers the 3rd, and the shouldPartialUnroll() covers the 5th. The output of each function, Optional<unsigned>, could be a value for UP.Count, which means unrolling factor has been set, or None, which means decision hasn't been made yet and should try the next priority.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106001/new/

https://reviews.llvm.org/D106001

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106001.362120.patch
Type: text/x-patch
Size: 16155 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210727/5dd11450/attachment.bin>


More information about the llvm-commits mailing list