Dialects include more than just ops, so this suffix is outdated. Follows
discussion in
https://llvm.discourse.group/t/rfc-canonical-file-paths-to-dialects/621
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/docs/Tutorials/CreatingADialect.md | ||
---|---|---|
29 | Side note: why is this file wrapped at 72 characters? |
I just got tripped up by the opacity of this today in npcomp. Thanks for making it more explicit. May want to give it a bit for others to respond.
What opacity? Meaning, this seems like changing one convention for another, not sure what it makes more transparent. Is there documentation of the convention missing? (It was inconsistent but I never relied on me typing these directly so probably this refers to not needing to look up things)
mlir/docs/Tutorials/CreatingADialect.md | ||
---|---|---|
41 | Did you miss this one? |
mlir/docs/Tutorials/CreatingADialect.md | ||
---|---|---|
41 | Unfortunately this actually references the td file which frequently is called FooOps :-/ There's some discussion in the discourse thread. We could make this example assume all one TD file rather than a FooOps.td file |
Sorry for the half comment. "This" was referring to the multi-step cleanup referenced on the originating Discourse post.
Side note: why is this file wrapped at 72 characters?