This details the structure of the format, it's requirements, and gives a few examples.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Unit tests: pass. 62423 tests passed, 0 failed and 845 were skipped.
clang-tidy: pass.
clang-format: pass.
Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml
Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.
Looks good thanks
mlir/docs/OpDefinitions.md | ||
---|---|---|
608 | Does this render correctly with hugo? | |
623 | It might be good to point out that this is a requirement of parsing custom forms in general and not specific to this. | |
631 | Are optional attributes handled? I'm assuming no special handling, only via attr-dict. | |
654–655 | Reflow? | |
1129 | Awh, I just looked at when this was originally written :) |
Resolve most comments.
mlir/docs/OpDefinitions.md | ||
---|---|---|
603 | I added, and linked to, "variable" to be a bit more specific. The operand/result here refers to the thing registered on the operation in ODS, so variadic isn't differentiated from non-variadic. | |
623 | Several of these technically apply to the custom form in general, i.e. you should really make sure all operands are covered. Is there a specific way that you suggest I point this out? | |
631 | Yep. There is no distinction between optional/non-optional attributes. We just elide any attributes that were explicitly present in the format from the attribute dictionary. | |
632 | See response to Jacques. |
Unit tests: pass. 62465 tests passed, 0 failed and 845 were skipped.
clang-tidy: pass.
clang-format: pass.
Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml
Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.
mlir/docs/OpDefinitions.md | ||
---|---|---|
608 | Confirmed that it does. |
"all the operands" to be clear.