[Mlir-commits] [mlir] [mlir] add a fluent API to GreedyRewriterConfig (PR #132253)

Mehdi Amini llvmlistbot at llvm.org
Thu Mar 20 10:06:20 PDT 2025


================
@@ -50,24 +50,41 @@ class GreedyRewriteConfig {
   ///
   /// Note: Only applicable when simplifying entire regions.
   bool useTopDownTraversal = false;
+  GreedyRewriteConfig &setUseTopDownTraversal(bool use = true) {
+    useTopDownTraversal = use;
+    return *this;
+  }
----------------
joker-eph wrote:

I'm all for encapsulation, but if we're adding methods, can we make all the data members private and turn this into a class?

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


More information about the Mlir-commits mailing list