Also, gen_spirv_dialect script was broken was broken due to SPV->SPIRV renamings.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| mlir/include/mlir/Dialect/SPIRV/IR/SPIRVGroupOps.td | ||
|---|---|---|
| 22 | Can you update the summary and description to match the extension spec wording? It's a bit annoying that the main SPIR-V spec only says "TBD" and no explanation so this is inheriting that. We'd prefer to have clear semantics here though. Diverging here is fine; the automatic importing tool is only meant for simplifying the process of defining new ops; some manual modification before/after is fine. | |
| 53 | This and the result should be changed to SPIRV_ScalarOrVectorOf<SPIRV_Float>? Similarly for the following places where we also use SPIRV_Scalar? | |
| mlir/include/mlir/Dialect/SPIRV/IR/SPIRVGroupOps.td | ||
|---|---|---|
| 53 | For non-KHR ops spec says Result Type must be a 16-bit, 32-bit, or 64-bit floating-point type scalar. The type of X must be the same as Result Type. Not vector, so I'll change them to just SPIRV_Float/SPIRV_Integer | |
Not vector, so I'll change them to just SPIRV_Float/SPIRV_Integer
Nevermind, looked in wrong place
Can you update the summary and description to match the extension spec wording?
It's a bit annoying that the main SPIR-V spec only says "TBD" and no explanation so this is inheriting that. We'd prefer to have clear semantics here though. Diverging here is fine; the automatic importing tool is only meant for simplifying the process of defining new ops; some manual modification before/after is fine.