[Mlir-commits] [mlir] [mlir][sparse] use shared value storage between wrapped iterator and the wrapper. (PR #80046)
    Aart Bik 
    llvmlistbot at llvm.org
       
    Tue Jan 30 11:32:52 PST 2024
    
    
  
================
@@ -219,7 +242,16 @@ class SparseIterator {
   // For trivial iterators, it is the position; for dedup iterators, it consists
   // of the positon and the segment high, for non-empty subsection iterator, it
   // is the metadata that specifies the subsection.
-  MutableArrayRef<Value> itVals;
+  // Note that the wrapped iterator shares the same storage to maintain itVals
+  // with it wrapper, which means the wrapped iterator might only owns a subset
----------------
aartbik wrote:
typo: might only own (no s)
https://github.com/llvm/llvm-project/pull/80046
    
    
More information about the Mlir-commits
mailing list