[Mlir-commits] [mlir] [mlir][side effect] refactor(*): Include more precise side effects (PR #94213)

[donald chen]

================
@@ -149,11 +149,20 @@ class EffectInstance {
                  Resource *resource = DefaultResource::get())
       : effect(effect), resource(resource), value(value), stage(0),
         effectOnFullRegion(false) {}
+  EffectInstance(EffectT *effect, OpOperand *opd,
----------------
cxy-1993 wrote:

As previously discussed, I have added OpResult and BlockArgument, and reimplemented it using templates. Please review again

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