Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/include/mlir/Dialect/SparseTensor/Utils/Merger.h | ||
---|---|---|
88 | to generate (no s) | |
mlir/lib/Dialect/SparseTensor/Transforms/Sparsification.cpp | ||
1147 | should we avoid reaching this altogether? I would hate for this to mess up with the semi-ring stuff.... | |
1150–1166 | I like the original v0/v1 a bit better, since lhs and rhs always reminds me of an assignment with lval and rval, and the vi corresponds to the ei in the children field | |
1160 | can this be an else-if or can both branches be taken?! | |
mlir/lib/Dialect/SparseTensor/Utils/Merger.cpp | ||
334 | can we reach this "recursively" or is this always called from top? zeroMapSet(.) and use that asserting on kCmpI/F and asserting on non-kCmpI/F here? That would make a bit less special case iffing. |
mlir/lib/Dialect/SparseTensor/Transforms/Sparsification.cpp | ||
---|---|---|
1137–1138 | remove empty line | |
1163 | why not have a third else { v0 = genExp v1 = genExp } | |
mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_cmp.mlir | ||
25 ↗ | (On Diff #531002) | remove |
89 ↗ | (On Diff #531002) | remove |
92 ↗ | (On Diff #531002) | rephrase: that constructs test matrices and calls.... |
122 ↗ | (On Diff #531002) | period at end |
to generate (no s)