[PATCH] D74938: [mlir][Tutorial] Add a section to Toy Ch.2 detailing the custom assembly format.

Alex Zinenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 05:11:53 PST 2020


ftynse added inline comments.


================
Comment at: mlir/docs/Tutorials/Toy/Ch-2.md:671
+custom format in C++. After beautifying the format of a few of our operations we
+now get a much more readable:
+
----------------
mehdi_amini wrote:
> Why not provide a custom syntax for all of them?
I think it is valuable to demonstrate the C++ way as well. Partly because the declarative format does not (and probably cannot) support all use cases, and partly to showcase the effort that went into building the infrastructure.

Personally, I really dislike the educational materials that don't offer enough explanation on what is going behind the nice and magic-feeling solution.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74938/new/

https://reviews.llvm.org/D74938





More information about the llvm-commits mailing list