[Mlir-commits] [mlir] [mlir] Notify rewriter listeners when hoisting constants (PR #233776)

[Mehdi Amini]

================
@@ -33,20 +34,19 @@ struct TestSingleFold : public PassWrapper<TestSingleFold, OperationPass<>>,
   StringRef getDescription() const final {
     return "Test single-pass operation folding and dead constant elimination";
   }
-  // All constants in the operation post folding.
-  SmallVector<Operation *> existingConstants;
+  // Moves can report the same constant more than once. Keep each constant
+  // once so that cleanup cannot visit an operation after erasing it.
+  llvm::SmallSetVector<Operation *, 8> existingConstants;
----------------
joker-eph wrote:

```suggestion
  SmallSetVector<Operation *> existingConstants;
```

Not clear why this was changed

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