[flang-commits] [flang] [mlir] [MLIR][LLVM] Fix inlining of a single block ending with unreachable (PR #122615)

[William Moses via flang-commits]

wsmoses wrote:

Yeah we can override processInlingBlocks, but I worry that creating a second fictitious block will break other parts of the infrastructure (which for example, iterate through the previously computed list).

my alternate proposal is I can change processInliningBlocks to return a book indicating whether the fast path is still permitted. Unfortunately this still requires a change in code (which now returns a bool).

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