[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