[clang] [analyzer] Mention possibility of underflow in array overflow errors (PR #84201)

Balázs Kéri via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 7 04:22:00 PST 2024


================
@@ -603,6 +611,8 @@ void ArrayBoundCheckerV2::performCheck(const Expr *E, CheckerContext &C) const {
     auto [WithinUpperBound, ExceedsUpperBound] =
         compareValueToThreshold(State, ByteOffset, *KnownSize, SVB);
 
+    bool AssumedNonNegative = SUR.assumedNonNegative();
----------------
balazske wrote:

The name `AssumedNonNegative` looks misleading (in variable and function arguments too). This has meaning like `CanBeNegative` which is a better name for this.

https://github.com/llvm/llvm-project/pull/84201


More information about the cfe-commits mailing list