[Mlir-commits] [mlir] [mlir][sparse] introduce `sparse_tensor.coiterate` operation. (PR #101100)
Ingo Müller
llvmlistbot at llvm.org
Wed Jul 31 03:03:06 PDT 2024
================
@@ -61,37 +61,62 @@ struct COOSegment {
/// A simple wrapper to encode a bitset of (at most 64) levels, currently used
/// by `sparse_tensor.iterate` operation for the set of levels on which the
/// coordinates should be loaded.
-class LevelSet {
----------------
ingomueller-net wrote:
Nit: I prefer multiple smaller PRs over single big ones. This change looks like a good candidate to factor out.
Don't bother changing this now, though! It's a (minor) comment to consider in the future...
https://github.com/llvm/llvm-project/pull/101100
More information about the Mlir-commits
mailing list