Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
roundtrip and validation tests? also makes it easier to see what it will look like
mlir/include/mlir/Dialect/SparseTensor/IR/SparseTensorAttrDefs.td | ||
---|---|---|
187 | This needs a lot more documentation what the triplets mean per dimension ;-) |
mlir/include/mlir/Dialect/SparseTensor/IR/SparseTensorAttrDefs.td | ||
---|---|---|
27 | We normally end a comment with a period, see also SparseTensorOps.td. | |
33–36 | Do we intend to add the description? | |
46 | offset/size/stride to be consistent with ordering of the parameters in the code? | |
47 | Would it be better if we use mlir::ShapedType::kDynamic instead? | |
mlir/lib/Dialect/SparseTensor/IR/SparseTensorDialect.cpp | ||
65 | Would it be better if we say "positive value or ? for" here? |
We normally end a comment with a period, see also SparseTensorOps.td.