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.