[clang] [analyzer] Print comparison literals with the right signedness (PR #219136)

Balázs Benics via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 27 01:48:36 PDT 2026


================
@@ -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


More information about the cfe-commits mailing list