[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