fix(hash): include tensor data pointer in InfiniOP cache keys

Ceng23333 · Ceng23333 · commit ef4c1504917b · 2026-04-17T03:34:59.000Z
Narrowed views can share shape/strides; hashing the resolved base address
avoids plan-cache collisions across different storage offsets.

Made-with: Cursor
diff --git a/include/infinicore/common/hash.hpp b/include/infinicore/common/hash.hpp
@@ -2,6 +2,7 @@
 
 #include "../tensor.hpp"
 
+#include <cstdint>
 #include <optional>
 #include <type_traits>
 
@@ -28,6 +29,9 @@ inline void hash_combine(size_t &seed, Tensor tensor) {
     for (Stride stride : tensor->strides()) {
         hash_combine(seed, static_cast<size_t>(stride));
     }
+    // Storage offset is not part of shape/strides but must differ across `narrow` views that share
+    // the same meta; include the resolved base pointer for InfiniOP / graph plan cache keys.
+    hash_combine(seed, static_cast<size_t>(reinterpret_cast<uintptr_t>(tensor->data())));
 }
 
 // Specialization for optional

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,7 @@`
`2`	`2`
`3`	`3`	`#include "../tensor.hpp"`
`4`	`4`
	`5`	`+#include <cstdint>`
`5`	`6`	`#include <optional>`
`6`	`7`	`#include <type_traits>`
`7`	`8`
`@@ -28,6 +29,9 @@ inline void hash_combine(size_t &seed, Tensor tensor) {`
`28`	`29`	`for (Stride stride : tensor->strides()) {`
`29`	`30`	`hash_combine(seed, static_cast<size_t>(stride));`
`30`	`31`	`}`
	`32`	+ // Storage offset is not part of shape/strides but must differ across `narrow` views that share
	`33`	`+ // the same meta; include the resolved base pointer for InfiniOP / graph plan cache keys.`
	`34`	`+ hash_combine(seed, static_cast<size_t>(reinterpret_cast<uintptr_t>(tensor->data())));`
`31`	`35`	`}`
`32`	`36`
`33`	`37`	`// Specialization for optional`