[Mlir-commits] [mlir] [mlir][Transforms][NFC] Dialect Conversion: Earlier `isIgnored` check (PR #148360)

[Markus Böck]

https://github.com/zero9178 approved this pull request.

The change itself looks good to me, although I am wondering whether the new implementation calling `legalize` with an erased op isn't an issue.

Once you erase an Op the address might be reused by new ops created afterwards making it not save to even do `contains` operations in a set using the pointer address. 

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