[PATCH] D156087: [MLIR] Add stage to side effect

[donald chen via Phabricator via llvm-commits]

cxy-1993 added a comment.

Thanks for your advice.
The "partial/full region" aspect of the effect  is also an important feature in side effect,  i have already added them to the member variable of EffectInstance and they can be specified via the 'td' parameter.
Regarding May/Must associated with the effect, I believe you are referring to the relationship between memrefs. If that's the case, we will leave this issue to the pass to invoke alias analysis. If it's not, please let me know.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156087/new/

https://reviews.llvm.org/D156087