diff --git a/clang/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp b/clang/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp --- a/clang/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp +++ b/clang/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp @@ -2134,6 +2134,23 @@ } }; +class ArrayIndexHandler final : public ExpressionHandler { +public: + using ExpressionHandler::ExpressionHandler; + + Tracker::Result handle(const Expr *Inner, const ExplodedNode *InputNode, + const ExplodedNode *LVNode, + TrackingOptions Opts) override { + // Track the index if this is an array subscript. + if (const auto *Arr = dyn_cast(Inner)) + return getParentTracker().track( + Arr->getIdx(), LVNode, + {Opts.Kind, /*EnableNullFPSuppression*/ false}); + + return {}; + } +}; + class DefaultExpressionHandler final : public ExpressionHandler { public: using ExpressionHandler::ExpressionHandler; @@ -2146,12 +2163,6 @@ PathSensitiveBugReport &Report = getParentTracker().getReport(); Tracker::Result Result; - // Track the index if this is an array subscript. - if (const auto *Arr = dyn_cast(Inner)) - Result.combineWith(getParentTracker().track( - Arr->getIdx(), LVNode, - {Opts.Kind, /*EnableNullFPSuppression*/ false})); - // See if the expression we're interested refers to a variable. // If so, we can track both its contents and constraints on its value. if (ExplodedGraph::isInterestingLValueExpr(Inner)) { @@ -2320,6 +2331,7 @@ // Default expression handlers. addLowPriorityHandler(); addLowPriorityHandler(); + addLowPriorityHandler(); addLowPriorityHandler(); addLowPriorityHandler(); // Default store handlers. @@ -2340,8 +2352,12 @@ // Iterate through the handlers in the order according to their priorities. for (ExpressionHandlerPtr &Handler : ExpressionHandlers) { CombinedResult.combineWith(Handler->handle(Inner, N, LVNode, Opts)); - if (CombinedResult.WasInterrupted) + if (CombinedResult.WasInterrupted) { + // There is no need to confuse our users here. + // We got interrupted, but our users don't need to know about it. + CombinedResult.WasInterrupted = false; break; + } } return CombinedResult;