[Mlir-commits] [mlir] [mlir] Add struct parsing and printing utilities (PR #133939)

Jorn Tuyls llvmlistbot at llvm.org
Thu Apr 3 15:25:55 PDT 2025


jtuyls wrote:

> Then in your code, you only have one custom directive (for dim parsing/printing), and your done (your attr parser/printer are now declarative). That feels much better, and for your case would remove a lot of the C++ (in your case, there are a few ops which would be simplified just from that).

Yes, this would remove a lot of code in most cases. Let me give this a shot.

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


More information about the Mlir-commits mailing list