Along the way, make the default pattern fail instead of crashing
when an elementwise op is not supported yet.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Just some nits
mlir/lib/Conversion/GPUToSPIRV/WmmaOpsToSPIRV.cpp | ||
---|---|---|
31 | nit: s/element wise/elementwise/ for consistency | |
32 | nit: createElementwiseOp to much the capitalization of SubgroupMmaElementwiseOp and other names in mlir? | |
208–209 | Maybe move this above the previous check? I'd assume that it's more useful to know that the number of operands is off. | |
215 | nit: can you split this into two separate definitions? | |
232 | nit: flip the if condition and use early return? IE auto cc = splat.get...; if (!cc) return failure() |
nit: s/element wise/elementwise/ for consistency