[llvm-branch-commits] [mlir] [draft] Dialect Conversion without Rollback (PR #93412)

[Oleksandr Alex Zinenko via llvm-branch-commits]

================
@@ -321,15 +323,15 @@ class RandomizedWorklist : public Worklist {
 /// to the worklist in the beginning.
 class GreedyPatternRewriteDriver : public RewriterBase::Listener {
 protected:
-  explicit GreedyPatternRewriteDriver(MLIRContext *ctx,
+  explicit GreedyPatternRewriteDriver(PatternRewriter &rewriter,
                                       const FrozenRewritePatternSet &patterns,
                                       const GreedyRewriteConfig &config);
 
   /// Add the given operation to the worklist.
   void addSingleOpToWorklist(Operation *op);
 
   /// Add the given operation and its ancestors to the worklist.
-  void addToWorklist(Operation *op);
+  virtual void addToWorklist(Operation *op);
----------------
ftynse wrote:

Do we have an estimate of how much overhead adding a vtable to this class introduces?

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