[PATCH] D91481: [LoopUnroll] Discount uniform instructions in cost models

Whitney Tsang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 1 05:12:30 PST 2020


Whitney added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp:769
+    // iterations.  We should break the backedge, not unroll that.
+    if (LocalUniformSize + UP.BEInsns + 1 > LoopSize)
+      LocalUniformSize = 0;
----------------
Do we need a test case for this case where the loop is not unrolled because the entire loop body is uniform?


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

https://reviews.llvm.org/D91481



More information about the llvm-commits mailing list