[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