[Mlir-commits] [mlir] [mlir][Transforms] Dialect conversion: Fix bug in `computeNecessaryMaterializations` (PR #104630)

[Billy Zhu]

zyx-billy wrote:

@matthias-springer I've been seeing the null operand bug too. Do you think this fixes it: https://github.com/llvm/llvm-project/pull/104648? It fixes the null operand issue, and exposes the new behavior of instead of leaving two inverse `unrealized_conversion_cast`s, now the second cast's result gets replaced with the input of the first cast, and the two casts are no longer needed.

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