[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