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

via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 1 03:40:53 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff d1538c15f9c65a70f4650bd724972536f00f5094 d966a0471e6507eafab7ee5d1bb10528ad41e767 -- llvm/lib/Transforms/Scalar/NewGVN.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/lib/Transforms/Scalar/NewGVN.cpp b/llvm/lib/Transforms/Scalar/NewGVN.cpp
index 0db4e4051e..cded0d543e 100644
--- a/llvm/lib/Transforms/Scalar/NewGVN.cpp
+++ b/llvm/lib/Transforms/Scalar/NewGVN.cpp
@@ -383,10 +383,9 @@ public:
     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, RepStoredValue, RepMemoryAccess) !=
+        std::tie(Other->StoreCount, Other->RepLeader, Other->RepStoredValue,
+                 Other->RepMemoryAccess))
       return false;
     if (DefiningExpr != Other->DefiningExpr)
       if (!DefiningExpr || !Other->DefiningExpr ||

``````````

</details>


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


More information about the llvm-commits mailing list