Memoize loc_t hashes

Signed-off-by: Kyle Cripps <[email protected]>
p4lang · Jul 17, 2024 · 50c0f9f · 50c0f9f
1 parent fead2eb
commit 50c0f9f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/frontends/p4/def_use.cpp b/frontends/p4/def_use.cpp
@@ -764,9 +764,13 @@ void ComputeWriteSet::visitVirtualMethods(const IR::IndexedVector<IR::Declaratio
 }
 
 std::size_t P4::loc_t::hash() const {
-    if (!parent) return Util::Hash{}(node->id);
-
-    return Util::Hash{}(node->id, parent->hash());
+    if (!computedHash) {
+        if (!parent)
+            computedHash = Util::Hash{}(node->id);
+        else
+            computedHash = Util::Hash{}(node->id, parent->hash());
+    }
+    return computedHash;
 }
 
 // Returns program location of n, given the program location of n's direct parent.

diff --git a/frontends/p4/def_use.h b/frontends/p4/def_use.h
@@ -39,6 +39,7 @@ class LocationSet;
 struct loc_t {
     const IR::Node *node;
     const loc_t *parent;
+    mutable std::size_t computedHash = 0;
     bool operator==(const loc_t &a) const {
         if (node != a.node) return false;
         if (parent == a.parent) return true;