[llvm] [GVNSink] Avoid repeated hash lookups (NFC) (PR #113023)

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 19 18:07:34 PDT 2024


================
@@ -535,14 +535,10 @@ class ValueTable {
     uint32_t e = ExpressionNumbering[exp];
     if (!e) {
       hash_code H = exp->getHashValue([=](Value *V) { return lookupOrAdd(V); });
-      auto I = HashNumbering.find(H);
-      if (I != HashNumbering.end()) {
-        e = I->second;
-      } else {
-        e = nextValueNumber++;
-        HashNumbering[H] = e;
-        ExpressionNumbering[exp] = e;
-      }
+      auto [I, Inserted] = HashNumbering.try_emplace(H, nextValueNumber + 1);
----------------
kazutakahirata wrote:

Oops.  Thanks for catching this!  Fixed in the latest iteration.

https://github.com/llvm/llvm-project/pull/113023


More information about the llvm-commits mailing list