[Mlir-commits] [mlir] [mlir][memref][nfc] push early-exit to earlier (PR #140730)
Andrzej Warzyński
llvmlistbot at llvm.org
Sat May 24 02:11:25 PDT 2025
banach-space wrote:
Hey @Garra1980 , thanks for posting this and apologies if you're experiencing issues due to this PR.
> What was the rationale behind this change?
Early exits are [encouraged in LLVM](https://llvm.org/docs/CodingStandards.html#use-early-exits-and-continue-to-simplify-code) and this looked like (and still does) like an NFC.
> no flow from %12 to memref_i8 which seems to be wrong
It's not quite clear to me what is wrong 😅 Could you compare _before_ and _after_ IR? If _after_ is indeed incorrect, we should include your example as a test and revert this change.
https://github.com/llvm/llvm-project/pull/140730
More information about the Mlir-commits
mailing list