[Mlir-commits] [mlir] Reland "[mlir][reducer] Add eraseRedundantBlocksInRegion and getSuccessorForwardOperands API to BranchOpInterface" (PR #189253)

[Mehdi Amini]

================

----------------
joker-eph wrote:

I'm not comfortable with a change to one of the core interfaces with a new interface method being added in a PR that is about mlir-reduce.   
I would really rather see such changes pulled in a PR of its own.

Here I have concerns with the default behavior, considering the method description.

Please revert this PR now and split this out.

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