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()