[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