[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