================
@@ -513,10 +513,10 @@ class ConditionBRVisitor final : public 
BugReporterVisitor {
   bool printValue(const Expr *CondVarExpr, raw_ostream &Out,
                   const ExplodedNode *N, bool TookTrue, bool IsAssuming);
 
-  bool patternMatch(const Expr *Ex, const Expr *ParentEx, raw_ostream &Out,
-                    BugReporterContext &BRC, PathSensitiveBugReport &R,
-                    const ExplodedNode *N, std::optional<bool> &prunable,
-                    bool IsSameFieldName);
+  bool patternMatch(const Expr *Ex, const Expr *ParentEx, const Expr *OtherEx,
+                    raw_ostream &Out, BugReporterContext &BRC,
+                    PathSensitiveBugReport &R, const ExplodedNode *N,
+                    std::optional<bool> &prunable, bool IsSameFieldName);
----------------
steakhal wrote:

I don't think its a good idea to mix other NFC changes but okay.
Done in 
https://github.com/llvm/llvm-project/pull/219136/commits/198b80c4f765a83143ab539b991b444e10a0db42

Afterall this wasn't really a huge pollution.

https://github.com/llvm/llvm-project/pull/219136
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to