[Mlir-commits] [mlir] [mlir][Transforms] Dialect conversion: Build unresolved materialization for replaced ops (PR #101514)

[Markus Böck]

zero9178 wrote:

> I think this might not work with graph regions... I'm trying to work up a simple example test case to show what I mean. I noticed this while integrating the latest MLIR in CIRCT, which makes heavy use of graph regions, and now has dialect conversion tests failing with null operands. I reverted this patch locally and those tests are passing again.

Your symptoms sound as if it is the same issue as was noted/addressed in https://github.com/llvm/llvm-project/pull/104630
Does that PR or the other referenced within fix it? 

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