[Mlir-commits] [mlir] [mlir][scf] add unroll-full option to test-loop-unrolling pass. (PR #127158)

[Oleksandr Alex Zinenko]

================
@@ -77,6 +82,9 @@ struct TestLoopUnrollingPass
                                 llvm::cl::init(false)};
   Option<unsigned> loopDepth{*this, "loop-depth", llvm::cl::desc("Loop depth."),
                              llvm::cl::init(0)};
+  Option<bool> unrollFull{*this, "unroll-full",
+                          llvm::cl::desc("Full unroll loops."),
+                          llvm::cl::init(false)};
----------------
ftynse wrote:

Nit: I'd just make the unroll-factor take -1 as "full" and avoid potential conflict when passing both an explicit factor and unroll-full=true.

https://github.com/llvm/llvm-project/pull/127158