[Mlir-commits] [mlir] [mlir][c] Change mlirOperationPrint API (PR #68066)

Jacques Pienaar llvmlistbot at llvm.org
Fri Oct 6 06:31:01 PDT 2023


jpienaar wrote:

Yes but we'd almost never want this form to be used. Currently one can simply pass a null to get same behavior.

The reason I made it breaking, is that the old print form should only ever be used if you are printing only one op and nothing else, but the cost of print iterating over the IR is hidden so its easy to miss and get O(n^2) behavior while the name makes this seem like a preferred print method.

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


More information about the Mlir-commits mailing list