Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| mlir/test/Dialect/SparseTensor/GPU/gpu_matmul_lib_2to4.mlir | ||
|---|---|---|
| 62 | I am not familiar with the attribute. Shall we use a flag like attr_name =1/0, or a flag where attr_name means set and its non-presence means not set? Is the latter possible in MLIR? | |
| mlir/test/Dialect/SparseTensor/GPU/gpu_matmul_lib_2to4.mlir | ||
|---|---|---|
| 62 | You can simply assign a string attribute to any MLIR op. So that would be a possible way until we have 2:4 properly defined into our sparse tensor type | |
| mlir/test/Dialect/SparseTensor/GPU/gpu_matmul_lib_2to4.mlir | ||
|---|---|---|
| 62 | SG! | |
| mlir/lib/Dialect/SparseTensor/Transforms/SparseGPUCodegen.cpp | ||
|---|---|---|
| 252 | can you move this to the "Library helper methods" where we have all the is/are tests on type also, this should probably use the isAllDense() or hasEncoding() utils from SparseTensorType | |
| 678 | remove comment altogether | |
| 1021 | use LLVM preferred style if (op->getAttr("DENSE24")) rewrite2To4SpMM(rewriter, op); rewriteSpMM(rewriter, op, enableRT); | |
| mlir/lib/Dialect/SparseTensor/Transforms/SparseGPUCodegen.cpp | ||
|---|---|---|
| 252 | Addressed. Please check if it works. | |
| mlir/lib/Dialect/SparseTensor/Transforms/SparseGPUCodegen.cpp | ||
|---|---|---|
| 40 | Oh, I meant to the section near L378, where the other type testers are and, can't this be simply return (getSparseTensorType(type).isAllDense()) or return (getSparseTensorType(type).hasEncoding() | |
Oh, I meant to the section near L378, where the other type testers are
and, can't this be simply
return (getSparseTensorType(type).isAllDense())
or
return (getSparseTensorType(type).hasEncoding()