[Mlir-commits] [mlir] [mlir] [nfc] add assert for function 'getReturnOps' (PR #120675)

donald chen llvmlistbot at llvm.org
Tue Jan 21 04:13:38 PST 2025


cxy-1993 wrote:

> Does this actually fix #120535 now? I was thinking of printing a proper error (`emitError`) instead of a crash due to failed assertion.

Emit error in this utility function is confusing. Do you mean that we should emitError at all call sites and abort the corresponding transform?

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


More information about the Mlir-commits mailing list