[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