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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
