[clang] [analyzer] In LivenessValues::equals also check liveBindings (PR #157645)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 9 04:01:03 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balazs Benics (steakhal)
<details>
<summary>Changes</summary>
This was likely accidentally omitted when `liveBindings` was introduced. I don't think in practice it matters.
---
Full diff: https://github.com/llvm/llvm-project/pull/157645.diff
1 Files Affected:
- (modified) clang/lib/Analysis/LiveVariables.cpp (+2-1)
``````````diff
diff --git a/clang/lib/Analysis/LiveVariables.cpp b/clang/lib/Analysis/LiveVariables.cpp
index 375fdb3695e2f..db88d4f7ee0f7 100644
--- a/clang/lib/Analysis/LiveVariables.cpp
+++ b/clang/lib/Analysis/LiveVariables.cpp
@@ -128,7 +128,8 @@ LiveVariablesImpl::merge(LiveVariables::LivenessValues valsA,
}
bool LiveVariables::LivenessValues::equals(const LivenessValues &V) const {
- return liveExprs == V.liveExprs && liveDecls == V.liveDecls;
+ return liveExprs == V.liveExprs && liveDecls == V.liveDecls &&
+ liveBindings == V.liveBindings;
}
//===----------------------------------------------------------------------===//
``````````
</details>
https://github.com/llvm/llvm-project/pull/157645
More information about the cfe-commits
mailing list