[llvm] [NewGVN] Prevent cyclic dependencies by ensuring Leader has min RPO number (PR #82110)

Nuno Lopes via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 18 05:00:15 PST 2024


================
@@ -374,9 +381,10 @@ class CongruenceClass {
     if (this == Other)
       return true;
 
-    if (std::tie(StoreCount, RepLeader, RepStoredValue, RepMemoryAccess) !=
-        std::tie(Other->StoreCount, Other->RepLeader, Other->RepStoredValue,
-                 Other->RepMemoryAccess))
+    if (std::tie(StoreCount, RepLeader.first, RepStoredValue,
----------------
nunoplopes wrote:

why not compare the whole RepLeader instead of just .first?

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


More information about the llvm-commits mailing list