1e5dd7070Spatrick //===- UninitializedValues.cpp - Find Uninitialized Values ----------------===//
2e5dd7070Spatrick //
3e5dd7070Spatrick // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4e5dd7070Spatrick // See https://llvm.org/LICENSE.txt for license information.
5e5dd7070Spatrick // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6e5dd7070Spatrick //
7e5dd7070Spatrick //===----------------------------------------------------------------------===//
8e5dd7070Spatrick //
9e5dd7070Spatrick // This file implements uninitialized values analysis for source-level CFGs.
10e5dd7070Spatrick //
11e5dd7070Spatrick //===----------------------------------------------------------------------===//
12e5dd7070Spatrick
13e5dd7070Spatrick #include "clang/Analysis/Analyses/UninitializedValues.h"
14e5dd7070Spatrick #include "clang/AST/Attr.h"
15e5dd7070Spatrick #include "clang/AST/Decl.h"
16e5dd7070Spatrick #include "clang/AST/DeclBase.h"
17e5dd7070Spatrick #include "clang/AST/Expr.h"
18e5dd7070Spatrick #include "clang/AST/OperationKinds.h"
19e5dd7070Spatrick #include "clang/AST/Stmt.h"
20e5dd7070Spatrick #include "clang/AST/StmtObjC.h"
21e5dd7070Spatrick #include "clang/AST/StmtVisitor.h"
22e5dd7070Spatrick #include "clang/AST/Type.h"
23e5dd7070Spatrick #include "clang/Analysis/Analyses/PostOrderCFGView.h"
24e5dd7070Spatrick #include "clang/Analysis/AnalysisDeclContext.h"
25e5dd7070Spatrick #include "clang/Analysis/CFG.h"
26e5dd7070Spatrick #include "clang/Analysis/DomainSpecific/ObjCNoReturn.h"
27ec727ea7Spatrick #include "clang/Analysis/FlowSensitive/DataflowWorklist.h"
28e5dd7070Spatrick #include "clang/Basic/LLVM.h"
29e5dd7070Spatrick #include "llvm/ADT/BitVector.h"
30e5dd7070Spatrick #include "llvm/ADT/DenseMap.h"
31e5dd7070Spatrick #include "llvm/ADT/PackedVector.h"
32e5dd7070Spatrick #include "llvm/ADT/SmallBitVector.h"
33e5dd7070Spatrick #include "llvm/ADT/SmallVector.h"
34e5dd7070Spatrick #include "llvm/Support/Casting.h"
35e5dd7070Spatrick #include <algorithm>
36e5dd7070Spatrick #include <cassert>
37*12c85518Srobert #include <optional>
38e5dd7070Spatrick
39e5dd7070Spatrick using namespace clang;
40e5dd7070Spatrick
41e5dd7070Spatrick #define DEBUG_LOGGING 0
42e5dd7070Spatrick
isTrackedVar(const VarDecl * vd,const DeclContext * dc)43e5dd7070Spatrick static bool isTrackedVar(const VarDecl *vd, const DeclContext *dc) {
44e5dd7070Spatrick if (vd->isLocalVarDecl() && !vd->hasGlobalStorage() &&
45e5dd7070Spatrick !vd->isExceptionVariable() && !vd->isInitCapture() &&
46e5dd7070Spatrick !vd->isImplicit() && vd->getDeclContext() == dc) {
47e5dd7070Spatrick QualType ty = vd->getType();
48*12c85518Srobert return ty->isScalarType() || ty->isVectorType() || ty->isRecordType() ||
49*12c85518Srobert ty->isRVVType();
50e5dd7070Spatrick }
51e5dd7070Spatrick return false;
52e5dd7070Spatrick }
53e5dd7070Spatrick
54e5dd7070Spatrick //------------------------------------------------------------------------====//
55e5dd7070Spatrick // DeclToIndex: a mapping from Decls we track to value indices.
56e5dd7070Spatrick //====------------------------------------------------------------------------//
57e5dd7070Spatrick
58e5dd7070Spatrick namespace {
59e5dd7070Spatrick
60e5dd7070Spatrick class DeclToIndex {
61e5dd7070Spatrick llvm::DenseMap<const VarDecl *, unsigned> map;
62e5dd7070Spatrick
63e5dd7070Spatrick public:
64e5dd7070Spatrick DeclToIndex() = default;
65e5dd7070Spatrick
66e5dd7070Spatrick /// Compute the actual mapping from declarations to bits.
67e5dd7070Spatrick void computeMap(const DeclContext &dc);
68e5dd7070Spatrick
69e5dd7070Spatrick /// Return the number of declarations in the map.
size() const70e5dd7070Spatrick unsigned size() const { return map.size(); }
71e5dd7070Spatrick
72e5dd7070Spatrick /// Returns the bit vector index for a given declaration.
73*12c85518Srobert std::optional<unsigned> getValueIndex(const VarDecl *d) const;
74e5dd7070Spatrick };
75e5dd7070Spatrick
76e5dd7070Spatrick } // namespace
77e5dd7070Spatrick
computeMap(const DeclContext & dc)78e5dd7070Spatrick void DeclToIndex::computeMap(const DeclContext &dc) {
79e5dd7070Spatrick unsigned count = 0;
80e5dd7070Spatrick DeclContext::specific_decl_iterator<VarDecl> I(dc.decls_begin()),
81e5dd7070Spatrick E(dc.decls_end());
82e5dd7070Spatrick for ( ; I != E; ++I) {
83e5dd7070Spatrick const VarDecl *vd = *I;
84e5dd7070Spatrick if (isTrackedVar(vd, &dc))
85e5dd7070Spatrick map[vd] = count++;
86e5dd7070Spatrick }
87e5dd7070Spatrick }
88e5dd7070Spatrick
getValueIndex(const VarDecl * d) const89*12c85518Srobert std::optional<unsigned> DeclToIndex::getValueIndex(const VarDecl *d) const {
90e5dd7070Spatrick llvm::DenseMap<const VarDecl *, unsigned>::const_iterator I = map.find(d);
91e5dd7070Spatrick if (I == map.end())
92*12c85518Srobert return std::nullopt;
93e5dd7070Spatrick return I->second;
94e5dd7070Spatrick }
95e5dd7070Spatrick
96e5dd7070Spatrick //------------------------------------------------------------------------====//
97e5dd7070Spatrick // CFGBlockValues: dataflow values for CFG blocks.
98e5dd7070Spatrick //====------------------------------------------------------------------------//
99e5dd7070Spatrick
100e5dd7070Spatrick // These values are defined in such a way that a merge can be done using
101e5dd7070Spatrick // a bitwise OR.
102e5dd7070Spatrick enum Value { Unknown = 0x0, /* 00 */
103e5dd7070Spatrick Initialized = 0x1, /* 01 */
104e5dd7070Spatrick Uninitialized = 0x2, /* 10 */
105e5dd7070Spatrick MayUninitialized = 0x3 /* 11 */ };
106e5dd7070Spatrick
isUninitialized(const Value v)107e5dd7070Spatrick static bool isUninitialized(const Value v) {
108e5dd7070Spatrick return v >= Uninitialized;
109e5dd7070Spatrick }
110e5dd7070Spatrick
isAlwaysUninit(const Value v)111e5dd7070Spatrick static bool isAlwaysUninit(const Value v) {
112e5dd7070Spatrick return v == Uninitialized;
113e5dd7070Spatrick }
114e5dd7070Spatrick
115e5dd7070Spatrick namespace {
116e5dd7070Spatrick
117e5dd7070Spatrick using ValueVector = llvm::PackedVector<Value, 2, llvm::SmallBitVector>;
118e5dd7070Spatrick
119e5dd7070Spatrick class CFGBlockValues {
120e5dd7070Spatrick const CFG &cfg;
121e5dd7070Spatrick SmallVector<ValueVector, 8> vals;
122e5dd7070Spatrick ValueVector scratch;
123e5dd7070Spatrick DeclToIndex declToIndex;
124e5dd7070Spatrick
125e5dd7070Spatrick public:
126e5dd7070Spatrick CFGBlockValues(const CFG &cfg);
127e5dd7070Spatrick
getNumEntries() const128e5dd7070Spatrick unsigned getNumEntries() const { return declToIndex.size(); }
129e5dd7070Spatrick
130e5dd7070Spatrick void computeSetOfDeclarations(const DeclContext &dc);
131e5dd7070Spatrick
getValueVector(const CFGBlock * block)132e5dd7070Spatrick ValueVector &getValueVector(const CFGBlock *block) {
133e5dd7070Spatrick return vals[block->getBlockID()];
134e5dd7070Spatrick }
135e5dd7070Spatrick
136e5dd7070Spatrick void setAllScratchValues(Value V);
137e5dd7070Spatrick void mergeIntoScratch(ValueVector const &source, bool isFirst);
138e5dd7070Spatrick bool updateValueVectorWithScratch(const CFGBlock *block);
139e5dd7070Spatrick
hasNoDeclarations() const140e5dd7070Spatrick bool hasNoDeclarations() const {
141e5dd7070Spatrick return declToIndex.size() == 0;
142e5dd7070Spatrick }
143e5dd7070Spatrick
144e5dd7070Spatrick void resetScratch();
145e5dd7070Spatrick
146e5dd7070Spatrick ValueVector::reference operator[](const VarDecl *vd);
147e5dd7070Spatrick
getValue(const CFGBlock * block,const CFGBlock * dstBlock,const VarDecl * vd)148e5dd7070Spatrick Value getValue(const CFGBlock *block, const CFGBlock *dstBlock,
149e5dd7070Spatrick const VarDecl *vd) {
150*12c85518Srobert std::optional<unsigned> idx = declToIndex.getValueIndex(vd);
151*12c85518Srobert return getValueVector(block)[*idx];
152e5dd7070Spatrick }
153e5dd7070Spatrick };
154e5dd7070Spatrick
155e5dd7070Spatrick } // namespace
156e5dd7070Spatrick
CFGBlockValues(const CFG & c)157e5dd7070Spatrick CFGBlockValues::CFGBlockValues(const CFG &c) : cfg(c), vals(0) {}
158e5dd7070Spatrick
computeSetOfDeclarations(const DeclContext & dc)159e5dd7070Spatrick void CFGBlockValues::computeSetOfDeclarations(const DeclContext &dc) {
160e5dd7070Spatrick declToIndex.computeMap(dc);
161e5dd7070Spatrick unsigned decls = declToIndex.size();
162e5dd7070Spatrick scratch.resize(decls);
163e5dd7070Spatrick unsigned n = cfg.getNumBlockIDs();
164e5dd7070Spatrick if (!n)
165e5dd7070Spatrick return;
166e5dd7070Spatrick vals.resize(n);
167e5dd7070Spatrick for (auto &val : vals)
168e5dd7070Spatrick val.resize(decls);
169e5dd7070Spatrick }
170e5dd7070Spatrick
171e5dd7070Spatrick #if DEBUG_LOGGING
printVector(const CFGBlock * block,ValueVector & bv,unsigned num)172e5dd7070Spatrick static void printVector(const CFGBlock *block, ValueVector &bv,
173e5dd7070Spatrick unsigned num) {
174e5dd7070Spatrick llvm::errs() << block->getBlockID() << " :";
175e5dd7070Spatrick for (const auto &i : bv)
176e5dd7070Spatrick llvm::errs() << ' ' << i;
177e5dd7070Spatrick llvm::errs() << " : " << num << '\n';
178e5dd7070Spatrick }
179e5dd7070Spatrick #endif
180e5dd7070Spatrick
setAllScratchValues(Value V)181e5dd7070Spatrick void CFGBlockValues::setAllScratchValues(Value V) {
182e5dd7070Spatrick for (unsigned I = 0, E = scratch.size(); I != E; ++I)
183e5dd7070Spatrick scratch[I] = V;
184e5dd7070Spatrick }
185e5dd7070Spatrick
mergeIntoScratch(ValueVector const & source,bool isFirst)186e5dd7070Spatrick void CFGBlockValues::mergeIntoScratch(ValueVector const &source,
187e5dd7070Spatrick bool isFirst) {
188e5dd7070Spatrick if (isFirst)
189e5dd7070Spatrick scratch = source;
190e5dd7070Spatrick else
191e5dd7070Spatrick scratch |= source;
192e5dd7070Spatrick }
193e5dd7070Spatrick
updateValueVectorWithScratch(const CFGBlock * block)194e5dd7070Spatrick bool CFGBlockValues::updateValueVectorWithScratch(const CFGBlock *block) {
195e5dd7070Spatrick ValueVector &dst = getValueVector(block);
196e5dd7070Spatrick bool changed = (dst != scratch);
197e5dd7070Spatrick if (changed)
198e5dd7070Spatrick dst = scratch;
199e5dd7070Spatrick #if DEBUG_LOGGING
200e5dd7070Spatrick printVector(block, scratch, 0);
201e5dd7070Spatrick #endif
202e5dd7070Spatrick return changed;
203e5dd7070Spatrick }
204e5dd7070Spatrick
resetScratch()205e5dd7070Spatrick void CFGBlockValues::resetScratch() {
206e5dd7070Spatrick scratch.reset();
207e5dd7070Spatrick }
208e5dd7070Spatrick
operator [](const VarDecl * vd)209e5dd7070Spatrick ValueVector::reference CFGBlockValues::operator[](const VarDecl *vd) {
210*12c85518Srobert return scratch[*declToIndex.getValueIndex(vd)];
211e5dd7070Spatrick }
212e5dd7070Spatrick
213e5dd7070Spatrick //------------------------------------------------------------------------====//
214e5dd7070Spatrick // Classification of DeclRefExprs as use or initialization.
215e5dd7070Spatrick //====------------------------------------------------------------------------//
216e5dd7070Spatrick
217e5dd7070Spatrick namespace {
218e5dd7070Spatrick
219e5dd7070Spatrick class FindVarResult {
220e5dd7070Spatrick const VarDecl *vd;
221e5dd7070Spatrick const DeclRefExpr *dr;
222e5dd7070Spatrick
223e5dd7070Spatrick public:
FindVarResult(const VarDecl * vd,const DeclRefExpr * dr)224e5dd7070Spatrick FindVarResult(const VarDecl *vd, const DeclRefExpr *dr) : vd(vd), dr(dr) {}
225e5dd7070Spatrick
getDeclRefExpr() const226e5dd7070Spatrick const DeclRefExpr *getDeclRefExpr() const { return dr; }
getDecl() const227e5dd7070Spatrick const VarDecl *getDecl() const { return vd; }
228e5dd7070Spatrick };
229e5dd7070Spatrick
230e5dd7070Spatrick } // namespace
231e5dd7070Spatrick
stripCasts(ASTContext & C,const Expr * Ex)232e5dd7070Spatrick static const Expr *stripCasts(ASTContext &C, const Expr *Ex) {
233e5dd7070Spatrick while (Ex) {
234e5dd7070Spatrick Ex = Ex->IgnoreParenNoopCasts(C);
235e5dd7070Spatrick if (const auto *CE = dyn_cast<CastExpr>(Ex)) {
236e5dd7070Spatrick if (CE->getCastKind() == CK_LValueBitCast) {
237e5dd7070Spatrick Ex = CE->getSubExpr();
238e5dd7070Spatrick continue;
239e5dd7070Spatrick }
240e5dd7070Spatrick }
241e5dd7070Spatrick break;
242e5dd7070Spatrick }
243e5dd7070Spatrick return Ex;
244e5dd7070Spatrick }
245e5dd7070Spatrick
246e5dd7070Spatrick /// If E is an expression comprising a reference to a single variable, find that
247e5dd7070Spatrick /// variable.
findVar(const Expr * E,const DeclContext * DC)248e5dd7070Spatrick static FindVarResult findVar(const Expr *E, const DeclContext *DC) {
249e5dd7070Spatrick if (const auto *DRE =
250e5dd7070Spatrick dyn_cast<DeclRefExpr>(stripCasts(DC->getParentASTContext(), E)))
251e5dd7070Spatrick if (const auto *VD = dyn_cast<VarDecl>(DRE->getDecl()))
252e5dd7070Spatrick if (isTrackedVar(VD, DC))
253e5dd7070Spatrick return FindVarResult(VD, DRE);
254e5dd7070Spatrick return FindVarResult(nullptr, nullptr);
255e5dd7070Spatrick }
256e5dd7070Spatrick
257e5dd7070Spatrick namespace {
258e5dd7070Spatrick
259e5dd7070Spatrick /// Classify each DeclRefExpr as an initialization or a use. Any
260e5dd7070Spatrick /// DeclRefExpr which isn't explicitly classified will be assumed to have
261e5dd7070Spatrick /// escaped the analysis and will be treated as an initialization.
262e5dd7070Spatrick class ClassifyRefs : public StmtVisitor<ClassifyRefs> {
263e5dd7070Spatrick public:
264e5dd7070Spatrick enum Class {
265e5dd7070Spatrick Init,
266e5dd7070Spatrick Use,
267e5dd7070Spatrick SelfInit,
268ec727ea7Spatrick ConstRefUse,
269e5dd7070Spatrick Ignore
270e5dd7070Spatrick };
271e5dd7070Spatrick
272e5dd7070Spatrick private:
273e5dd7070Spatrick const DeclContext *DC;
274e5dd7070Spatrick llvm::DenseMap<const DeclRefExpr *, Class> Classification;
275e5dd7070Spatrick
isTrackedVar(const VarDecl * VD) const276e5dd7070Spatrick bool isTrackedVar(const VarDecl *VD) const {
277e5dd7070Spatrick return ::isTrackedVar(VD, DC);
278e5dd7070Spatrick }
279e5dd7070Spatrick
280e5dd7070Spatrick void classify(const Expr *E, Class C);
281e5dd7070Spatrick
282e5dd7070Spatrick public:
ClassifyRefs(AnalysisDeclContext & AC)283e5dd7070Spatrick ClassifyRefs(AnalysisDeclContext &AC) : DC(cast<DeclContext>(AC.getDecl())) {}
284e5dd7070Spatrick
285e5dd7070Spatrick void VisitDeclStmt(DeclStmt *DS);
286e5dd7070Spatrick void VisitUnaryOperator(UnaryOperator *UO);
287e5dd7070Spatrick void VisitBinaryOperator(BinaryOperator *BO);
288e5dd7070Spatrick void VisitCallExpr(CallExpr *CE);
289e5dd7070Spatrick void VisitCastExpr(CastExpr *CE);
290e5dd7070Spatrick void VisitOMPExecutableDirective(OMPExecutableDirective *ED);
291e5dd7070Spatrick
operator ()(Stmt * S)292e5dd7070Spatrick void operator()(Stmt *S) { Visit(S); }
293e5dd7070Spatrick
get(const DeclRefExpr * DRE) const294e5dd7070Spatrick Class get(const DeclRefExpr *DRE) const {
295e5dd7070Spatrick llvm::DenseMap<const DeclRefExpr*, Class>::const_iterator I
296e5dd7070Spatrick = Classification.find(DRE);
297e5dd7070Spatrick if (I != Classification.end())
298e5dd7070Spatrick return I->second;
299e5dd7070Spatrick
300e5dd7070Spatrick const auto *VD = dyn_cast<VarDecl>(DRE->getDecl());
301e5dd7070Spatrick if (!VD || !isTrackedVar(VD))
302e5dd7070Spatrick return Ignore;
303e5dd7070Spatrick
304e5dd7070Spatrick return Init;
305e5dd7070Spatrick }
306e5dd7070Spatrick };
307e5dd7070Spatrick
308e5dd7070Spatrick } // namespace
309e5dd7070Spatrick
getSelfInitExpr(VarDecl * VD)310e5dd7070Spatrick static const DeclRefExpr *getSelfInitExpr(VarDecl *VD) {
311e5dd7070Spatrick if (VD->getType()->isRecordType())
312e5dd7070Spatrick return nullptr;
313e5dd7070Spatrick if (Expr *Init = VD->getInit()) {
314e5dd7070Spatrick const auto *DRE =
315e5dd7070Spatrick dyn_cast<DeclRefExpr>(stripCasts(VD->getASTContext(), Init));
316e5dd7070Spatrick if (DRE && DRE->getDecl() == VD)
317e5dd7070Spatrick return DRE;
318e5dd7070Spatrick }
319e5dd7070Spatrick return nullptr;
320e5dd7070Spatrick }
321e5dd7070Spatrick
classify(const Expr * E,Class C)322e5dd7070Spatrick void ClassifyRefs::classify(const Expr *E, Class C) {
323e5dd7070Spatrick // The result of a ?: could also be an lvalue.
324e5dd7070Spatrick E = E->IgnoreParens();
325e5dd7070Spatrick if (const auto *CO = dyn_cast<ConditionalOperator>(E)) {
326e5dd7070Spatrick classify(CO->getTrueExpr(), C);
327e5dd7070Spatrick classify(CO->getFalseExpr(), C);
328e5dd7070Spatrick return;
329e5dd7070Spatrick }
330e5dd7070Spatrick
331e5dd7070Spatrick if (const auto *BCO = dyn_cast<BinaryConditionalOperator>(E)) {
332e5dd7070Spatrick classify(BCO->getFalseExpr(), C);
333e5dd7070Spatrick return;
334e5dd7070Spatrick }
335e5dd7070Spatrick
336e5dd7070Spatrick if (const auto *OVE = dyn_cast<OpaqueValueExpr>(E)) {
337e5dd7070Spatrick classify(OVE->getSourceExpr(), C);
338e5dd7070Spatrick return;
339e5dd7070Spatrick }
340e5dd7070Spatrick
341e5dd7070Spatrick if (const auto *ME = dyn_cast<MemberExpr>(E)) {
342e5dd7070Spatrick if (const auto *VD = dyn_cast<VarDecl>(ME->getMemberDecl())) {
343e5dd7070Spatrick if (!VD->isStaticDataMember())
344e5dd7070Spatrick classify(ME->getBase(), C);
345e5dd7070Spatrick }
346e5dd7070Spatrick return;
347e5dd7070Spatrick }
348e5dd7070Spatrick
349e5dd7070Spatrick if (const auto *BO = dyn_cast<BinaryOperator>(E)) {
350e5dd7070Spatrick switch (BO->getOpcode()) {
351e5dd7070Spatrick case BO_PtrMemD:
352e5dd7070Spatrick case BO_PtrMemI:
353e5dd7070Spatrick classify(BO->getLHS(), C);
354e5dd7070Spatrick return;
355e5dd7070Spatrick case BO_Comma:
356e5dd7070Spatrick classify(BO->getRHS(), C);
357e5dd7070Spatrick return;
358e5dd7070Spatrick default:
359e5dd7070Spatrick return;
360e5dd7070Spatrick }
361e5dd7070Spatrick }
362e5dd7070Spatrick
363e5dd7070Spatrick FindVarResult Var = findVar(E, DC);
364e5dd7070Spatrick if (const DeclRefExpr *DRE = Var.getDeclRefExpr())
365e5dd7070Spatrick Classification[DRE] = std::max(Classification[DRE], C);
366e5dd7070Spatrick }
367e5dd7070Spatrick
VisitDeclStmt(DeclStmt * DS)368e5dd7070Spatrick void ClassifyRefs::VisitDeclStmt(DeclStmt *DS) {
369e5dd7070Spatrick for (auto *DI : DS->decls()) {
370e5dd7070Spatrick auto *VD = dyn_cast<VarDecl>(DI);
371e5dd7070Spatrick if (VD && isTrackedVar(VD))
372e5dd7070Spatrick if (const DeclRefExpr *DRE = getSelfInitExpr(VD))
373e5dd7070Spatrick Classification[DRE] = SelfInit;
374e5dd7070Spatrick }
375e5dd7070Spatrick }
376e5dd7070Spatrick
VisitBinaryOperator(BinaryOperator * BO)377e5dd7070Spatrick void ClassifyRefs::VisitBinaryOperator(BinaryOperator *BO) {
378e5dd7070Spatrick // Ignore the evaluation of a DeclRefExpr on the LHS of an assignment. If this
379e5dd7070Spatrick // is not a compound-assignment, we will treat it as initializing the variable
380e5dd7070Spatrick // when TransferFunctions visits it. A compound-assignment does not affect
381e5dd7070Spatrick // whether a variable is uninitialized, and there's no point counting it as a
382e5dd7070Spatrick // use.
383e5dd7070Spatrick if (BO->isCompoundAssignmentOp())
384e5dd7070Spatrick classify(BO->getLHS(), Use);
385e5dd7070Spatrick else if (BO->getOpcode() == BO_Assign || BO->getOpcode() == BO_Comma)
386e5dd7070Spatrick classify(BO->getLHS(), Ignore);
387e5dd7070Spatrick }
388e5dd7070Spatrick
VisitUnaryOperator(UnaryOperator * UO)389e5dd7070Spatrick void ClassifyRefs::VisitUnaryOperator(UnaryOperator *UO) {
390e5dd7070Spatrick // Increment and decrement are uses despite there being no lvalue-to-rvalue
391e5dd7070Spatrick // conversion.
392e5dd7070Spatrick if (UO->isIncrementDecrementOp())
393e5dd7070Spatrick classify(UO->getSubExpr(), Use);
394e5dd7070Spatrick }
395e5dd7070Spatrick
VisitOMPExecutableDirective(OMPExecutableDirective * ED)396e5dd7070Spatrick void ClassifyRefs::VisitOMPExecutableDirective(OMPExecutableDirective *ED) {
397e5dd7070Spatrick for (Stmt *S : OMPExecutableDirective::used_clauses_children(ED->clauses()))
398e5dd7070Spatrick classify(cast<Expr>(S), Use);
399e5dd7070Spatrick }
400e5dd7070Spatrick
isPointerToConst(const QualType & QT)401e5dd7070Spatrick static bool isPointerToConst(const QualType &QT) {
402e5dd7070Spatrick return QT->isAnyPointerType() && QT->getPointeeType().isConstQualified();
403e5dd7070Spatrick }
404e5dd7070Spatrick
hasTrivialBody(CallExpr * CE)405ec727ea7Spatrick static bool hasTrivialBody(CallExpr *CE) {
406ec727ea7Spatrick if (FunctionDecl *FD = CE->getDirectCallee()) {
407ec727ea7Spatrick if (FunctionTemplateDecl *FTD = FD->getPrimaryTemplate())
408ec727ea7Spatrick return FTD->getTemplatedDecl()->hasTrivialBody();
409ec727ea7Spatrick return FD->hasTrivialBody();
410ec727ea7Spatrick }
411ec727ea7Spatrick return false;
412ec727ea7Spatrick }
413ec727ea7Spatrick
VisitCallExpr(CallExpr * CE)414e5dd7070Spatrick void ClassifyRefs::VisitCallExpr(CallExpr *CE) {
415e5dd7070Spatrick // Classify arguments to std::move as used.
416e5dd7070Spatrick if (CE->isCallToStdMove()) {
417e5dd7070Spatrick // RecordTypes are handled in SemaDeclCXX.cpp.
418e5dd7070Spatrick if (!CE->getArg(0)->getType()->isRecordType())
419e5dd7070Spatrick classify(CE->getArg(0), Use);
420e5dd7070Spatrick return;
421e5dd7070Spatrick }
422ec727ea7Spatrick bool isTrivialBody = hasTrivialBody(CE);
423ec727ea7Spatrick // If a value is passed by const pointer to a function,
424e5dd7070Spatrick // we should not assume that it is initialized by the call, and we
425e5dd7070Spatrick // conservatively do not assume that it is used.
426ec727ea7Spatrick // If a value is passed by const reference to a function,
427ec727ea7Spatrick // it should already be initialized.
428e5dd7070Spatrick for (CallExpr::arg_iterator I = CE->arg_begin(), E = CE->arg_end();
429e5dd7070Spatrick I != E; ++I) {
430e5dd7070Spatrick if ((*I)->isGLValue()) {
431e5dd7070Spatrick if ((*I)->getType().isConstQualified())
432ec727ea7Spatrick classify((*I), isTrivialBody ? Ignore : ConstRefUse);
433e5dd7070Spatrick } else if (isPointerToConst((*I)->getType())) {
434e5dd7070Spatrick const Expr *Ex = stripCasts(DC->getParentASTContext(), *I);
435e5dd7070Spatrick const auto *UO = dyn_cast<UnaryOperator>(Ex);
436e5dd7070Spatrick if (UO && UO->getOpcode() == UO_AddrOf)
437e5dd7070Spatrick Ex = UO->getSubExpr();
438e5dd7070Spatrick classify(Ex, Ignore);
439e5dd7070Spatrick }
440e5dd7070Spatrick }
441e5dd7070Spatrick }
442e5dd7070Spatrick
VisitCastExpr(CastExpr * CE)443e5dd7070Spatrick void ClassifyRefs::VisitCastExpr(CastExpr *CE) {
444e5dd7070Spatrick if (CE->getCastKind() == CK_LValueToRValue)
445e5dd7070Spatrick classify(CE->getSubExpr(), Use);
446e5dd7070Spatrick else if (const auto *CSE = dyn_cast<CStyleCastExpr>(CE)) {
447e5dd7070Spatrick if (CSE->getType()->isVoidType()) {
448e5dd7070Spatrick // Squelch any detected load of an uninitialized value if
449e5dd7070Spatrick // we cast it to void.
450e5dd7070Spatrick // e.g. (void) x;
451e5dd7070Spatrick classify(CSE->getSubExpr(), Ignore);
452e5dd7070Spatrick }
453e5dd7070Spatrick }
454e5dd7070Spatrick }
455e5dd7070Spatrick
456e5dd7070Spatrick //------------------------------------------------------------------------====//
457e5dd7070Spatrick // Transfer function for uninitialized values analysis.
458e5dd7070Spatrick //====------------------------------------------------------------------------//
459e5dd7070Spatrick
460e5dd7070Spatrick namespace {
461e5dd7070Spatrick
462e5dd7070Spatrick class TransferFunctions : public StmtVisitor<TransferFunctions> {
463e5dd7070Spatrick CFGBlockValues &vals;
464e5dd7070Spatrick const CFG &cfg;
465e5dd7070Spatrick const CFGBlock *block;
466e5dd7070Spatrick AnalysisDeclContext ∾
467e5dd7070Spatrick const ClassifyRefs &classification;
468e5dd7070Spatrick ObjCNoReturn objCNoRet;
469e5dd7070Spatrick UninitVariablesHandler &handler;
470e5dd7070Spatrick
471e5dd7070Spatrick public:
TransferFunctions(CFGBlockValues & vals,const CFG & cfg,const CFGBlock * block,AnalysisDeclContext & ac,const ClassifyRefs & classification,UninitVariablesHandler & handler)472e5dd7070Spatrick TransferFunctions(CFGBlockValues &vals, const CFG &cfg,
473e5dd7070Spatrick const CFGBlock *block, AnalysisDeclContext &ac,
474e5dd7070Spatrick const ClassifyRefs &classification,
475e5dd7070Spatrick UninitVariablesHandler &handler)
476e5dd7070Spatrick : vals(vals), cfg(cfg), block(block), ac(ac),
477e5dd7070Spatrick classification(classification), objCNoRet(ac.getASTContext()),
478e5dd7070Spatrick handler(handler) {}
479e5dd7070Spatrick
480e5dd7070Spatrick void reportUse(const Expr *ex, const VarDecl *vd);
481ec727ea7Spatrick void reportConstRefUse(const Expr *ex, const VarDecl *vd);
482e5dd7070Spatrick
483e5dd7070Spatrick void VisitBinaryOperator(BinaryOperator *bo);
484e5dd7070Spatrick void VisitBlockExpr(BlockExpr *be);
485e5dd7070Spatrick void VisitCallExpr(CallExpr *ce);
486e5dd7070Spatrick void VisitDeclRefExpr(DeclRefExpr *dr);
487e5dd7070Spatrick void VisitDeclStmt(DeclStmt *ds);
488ec727ea7Spatrick void VisitGCCAsmStmt(GCCAsmStmt *as);
489e5dd7070Spatrick void VisitObjCForCollectionStmt(ObjCForCollectionStmt *FS);
490e5dd7070Spatrick void VisitObjCMessageExpr(ObjCMessageExpr *ME);
491e5dd7070Spatrick void VisitOMPExecutableDirective(OMPExecutableDirective *ED);
492e5dd7070Spatrick
isTrackedVar(const VarDecl * vd)493e5dd7070Spatrick bool isTrackedVar(const VarDecl *vd) {
494e5dd7070Spatrick return ::isTrackedVar(vd, cast<DeclContext>(ac.getDecl()));
495e5dd7070Spatrick }
496e5dd7070Spatrick
findVar(const Expr * ex)497e5dd7070Spatrick FindVarResult findVar(const Expr *ex) {
498e5dd7070Spatrick return ::findVar(ex, cast<DeclContext>(ac.getDecl()));
499e5dd7070Spatrick }
500e5dd7070Spatrick
getUninitUse(const Expr * ex,const VarDecl * vd,Value v)501e5dd7070Spatrick UninitUse getUninitUse(const Expr *ex, const VarDecl *vd, Value v) {
502e5dd7070Spatrick UninitUse Use(ex, isAlwaysUninit(v));
503e5dd7070Spatrick
504e5dd7070Spatrick assert(isUninitialized(v));
505e5dd7070Spatrick if (Use.getKind() == UninitUse::Always)
506e5dd7070Spatrick return Use;
507e5dd7070Spatrick
508e5dd7070Spatrick // If an edge which leads unconditionally to this use did not initialize
509e5dd7070Spatrick // the variable, we can say something stronger than 'may be uninitialized':
510e5dd7070Spatrick // we can say 'either it's used uninitialized or you have dead code'.
511e5dd7070Spatrick //
512e5dd7070Spatrick // We track the number of successors of a node which have been visited, and
513e5dd7070Spatrick // visit a node once we have visited all of its successors. Only edges where
514e5dd7070Spatrick // the variable might still be uninitialized are followed. Since a variable
515e5dd7070Spatrick // can't transfer from being initialized to being uninitialized, this will
516e5dd7070Spatrick // trace out the subgraph which inevitably leads to the use and does not
517e5dd7070Spatrick // initialize the variable. We do not want to skip past loops, since their
518e5dd7070Spatrick // non-termination might be correlated with the initialization condition.
519e5dd7070Spatrick //
520e5dd7070Spatrick // For example:
521e5dd7070Spatrick //
522e5dd7070Spatrick // void f(bool a, bool b) {
523e5dd7070Spatrick // block1: int n;
524e5dd7070Spatrick // if (a) {
525e5dd7070Spatrick // block2: if (b)
526e5dd7070Spatrick // block3: n = 1;
527e5dd7070Spatrick // block4: } else if (b) {
528e5dd7070Spatrick // block5: while (!a) {
529e5dd7070Spatrick // block6: do_work(&a);
530e5dd7070Spatrick // n = 2;
531e5dd7070Spatrick // }
532e5dd7070Spatrick // }
533e5dd7070Spatrick // block7: if (a)
534e5dd7070Spatrick // block8: g();
535e5dd7070Spatrick // block9: return n;
536e5dd7070Spatrick // }
537e5dd7070Spatrick //
538e5dd7070Spatrick // Starting from the maybe-uninitialized use in block 9:
539e5dd7070Spatrick // * Block 7 is not visited because we have only visited one of its two
540e5dd7070Spatrick // successors.
541e5dd7070Spatrick // * Block 8 is visited because we've visited its only successor.
542e5dd7070Spatrick // From block 8:
543e5dd7070Spatrick // * Block 7 is visited because we've now visited both of its successors.
544e5dd7070Spatrick // From block 7:
545e5dd7070Spatrick // * Blocks 1, 2, 4, 5, and 6 are not visited because we didn't visit all
546e5dd7070Spatrick // of their successors (we didn't visit 4, 3, 5, 6, and 5, respectively).
547e5dd7070Spatrick // * Block 3 is not visited because it initializes 'n'.
548e5dd7070Spatrick // Now the algorithm terminates, having visited blocks 7 and 8, and having
549e5dd7070Spatrick // found the frontier is blocks 2, 4, and 5.
550e5dd7070Spatrick //
551e5dd7070Spatrick // 'n' is definitely uninitialized for two edges into block 7 (from blocks 2
552e5dd7070Spatrick // and 4), so we report that any time either of those edges is taken (in
553e5dd7070Spatrick // each case when 'b == false'), 'n' is used uninitialized.
554e5dd7070Spatrick SmallVector<const CFGBlock*, 32> Queue;
555e5dd7070Spatrick SmallVector<unsigned, 32> SuccsVisited(cfg.getNumBlockIDs(), 0);
556e5dd7070Spatrick Queue.push_back(block);
557e5dd7070Spatrick // Specify that we've already visited all successors of the starting block.
558e5dd7070Spatrick // This has the dual purpose of ensuring we never add it to the queue, and
559e5dd7070Spatrick // of marking it as not being a candidate element of the frontier.
560e5dd7070Spatrick SuccsVisited[block->getBlockID()] = block->succ_size();
561e5dd7070Spatrick while (!Queue.empty()) {
562e5dd7070Spatrick const CFGBlock *B = Queue.pop_back_val();
563e5dd7070Spatrick
564e5dd7070Spatrick // If the use is always reached from the entry block, make a note of that.
565e5dd7070Spatrick if (B == &cfg.getEntry())
566e5dd7070Spatrick Use.setUninitAfterCall();
567e5dd7070Spatrick
568e5dd7070Spatrick for (CFGBlock::const_pred_iterator I = B->pred_begin(), E = B->pred_end();
569e5dd7070Spatrick I != E; ++I) {
570e5dd7070Spatrick const CFGBlock *Pred = *I;
571e5dd7070Spatrick if (!Pred)
572e5dd7070Spatrick continue;
573e5dd7070Spatrick
574e5dd7070Spatrick Value AtPredExit = vals.getValue(Pred, B, vd);
575e5dd7070Spatrick if (AtPredExit == Initialized)
576e5dd7070Spatrick // This block initializes the variable.
577e5dd7070Spatrick continue;
578e5dd7070Spatrick if (AtPredExit == MayUninitialized &&
579e5dd7070Spatrick vals.getValue(B, nullptr, vd) == Uninitialized) {
580e5dd7070Spatrick // This block declares the variable (uninitialized), and is reachable
581e5dd7070Spatrick // from a block that initializes the variable. We can't guarantee to
582e5dd7070Spatrick // give an earlier location for the diagnostic (and it appears that
583e5dd7070Spatrick // this code is intended to be reachable) so give a diagnostic here
584e5dd7070Spatrick // and go no further down this path.
585e5dd7070Spatrick Use.setUninitAfterDecl();
586e5dd7070Spatrick continue;
587e5dd7070Spatrick }
588e5dd7070Spatrick
589ec727ea7Spatrick if (AtPredExit == MayUninitialized) {
590ec727ea7Spatrick // If the predecessor's terminator is an "asm goto" that initializes
591*12c85518Srobert // the variable, then don't count it as "initialized" on the indirect
592*12c85518Srobert // paths.
593ec727ea7Spatrick CFGTerminator term = Pred->getTerminator();
594ec727ea7Spatrick if (const auto *as = dyn_cast_or_null<GCCAsmStmt>(term.getStmt())) {
595ec727ea7Spatrick const CFGBlock *fallthrough = *Pred->succ_begin();
596ec727ea7Spatrick if (as->isAsmGoto() &&
597ec727ea7Spatrick llvm::any_of(as->outputs(), [&](const Expr *output) {
598ec727ea7Spatrick return vd == findVar(output).getDecl() &&
599ec727ea7Spatrick llvm::any_of(as->labels(),
600ec727ea7Spatrick [&](const AddrLabelExpr *label) {
601ec727ea7Spatrick return label->getLabel()->getStmt() == B->Label &&
602ec727ea7Spatrick B != fallthrough;
603ec727ea7Spatrick });
604ec727ea7Spatrick })) {
605ec727ea7Spatrick Use.setUninitAfterDecl();
606ec727ea7Spatrick continue;
607ec727ea7Spatrick }
608ec727ea7Spatrick }
609ec727ea7Spatrick }
610ec727ea7Spatrick
611e5dd7070Spatrick unsigned &SV = SuccsVisited[Pred->getBlockID()];
612e5dd7070Spatrick if (!SV) {
613e5dd7070Spatrick // When visiting the first successor of a block, mark all NULL
614e5dd7070Spatrick // successors as having been visited.
615e5dd7070Spatrick for (CFGBlock::const_succ_iterator SI = Pred->succ_begin(),
616e5dd7070Spatrick SE = Pred->succ_end();
617e5dd7070Spatrick SI != SE; ++SI)
618e5dd7070Spatrick if (!*SI)
619e5dd7070Spatrick ++SV;
620e5dd7070Spatrick }
621e5dd7070Spatrick
622e5dd7070Spatrick if (++SV == Pred->succ_size())
623e5dd7070Spatrick // All paths from this block lead to the use and don't initialize the
624e5dd7070Spatrick // variable.
625e5dd7070Spatrick Queue.push_back(Pred);
626e5dd7070Spatrick }
627e5dd7070Spatrick }
628e5dd7070Spatrick
629e5dd7070Spatrick // Scan the frontier, looking for blocks where the variable was
630e5dd7070Spatrick // uninitialized.
631e5dd7070Spatrick for (const auto *Block : cfg) {
632e5dd7070Spatrick unsigned BlockID = Block->getBlockID();
633e5dd7070Spatrick const Stmt *Term = Block->getTerminatorStmt();
634e5dd7070Spatrick if (SuccsVisited[BlockID] && SuccsVisited[BlockID] < Block->succ_size() &&
635e5dd7070Spatrick Term) {
636e5dd7070Spatrick // This block inevitably leads to the use. If we have an edge from here
637e5dd7070Spatrick // to a post-dominator block, and the variable is uninitialized on that
638e5dd7070Spatrick // edge, we have found a bug.
639e5dd7070Spatrick for (CFGBlock::const_succ_iterator I = Block->succ_begin(),
640e5dd7070Spatrick E = Block->succ_end(); I != E; ++I) {
641e5dd7070Spatrick const CFGBlock *Succ = *I;
642e5dd7070Spatrick if (Succ && SuccsVisited[Succ->getBlockID()] >= Succ->succ_size() &&
643e5dd7070Spatrick vals.getValue(Block, Succ, vd) == Uninitialized) {
644e5dd7070Spatrick // Switch cases are a special case: report the label to the caller
645e5dd7070Spatrick // as the 'terminator', not the switch statement itself. Suppress
646e5dd7070Spatrick // situations where no label matched: we can't be sure that's
647e5dd7070Spatrick // possible.
648e5dd7070Spatrick if (isa<SwitchStmt>(Term)) {
649e5dd7070Spatrick const Stmt *Label = Succ->getLabel();
650e5dd7070Spatrick if (!Label || !isa<SwitchCase>(Label))
651e5dd7070Spatrick // Might not be possible.
652e5dd7070Spatrick continue;
653e5dd7070Spatrick UninitUse::Branch Branch;
654e5dd7070Spatrick Branch.Terminator = Label;
655e5dd7070Spatrick Branch.Output = 0; // Ignored.
656e5dd7070Spatrick Use.addUninitBranch(Branch);
657e5dd7070Spatrick } else {
658e5dd7070Spatrick UninitUse::Branch Branch;
659e5dd7070Spatrick Branch.Terminator = Term;
660e5dd7070Spatrick Branch.Output = I - Block->succ_begin();
661e5dd7070Spatrick Use.addUninitBranch(Branch);
662e5dd7070Spatrick }
663e5dd7070Spatrick }
664e5dd7070Spatrick }
665e5dd7070Spatrick }
666e5dd7070Spatrick }
667e5dd7070Spatrick
668e5dd7070Spatrick return Use;
669e5dd7070Spatrick }
670e5dd7070Spatrick };
671e5dd7070Spatrick
672e5dd7070Spatrick } // namespace
673e5dd7070Spatrick
reportUse(const Expr * ex,const VarDecl * vd)674e5dd7070Spatrick void TransferFunctions::reportUse(const Expr *ex, const VarDecl *vd) {
675e5dd7070Spatrick Value v = vals[vd];
676e5dd7070Spatrick if (isUninitialized(v))
677e5dd7070Spatrick handler.handleUseOfUninitVariable(vd, getUninitUse(ex, vd, v));
678e5dd7070Spatrick }
679e5dd7070Spatrick
reportConstRefUse(const Expr * ex,const VarDecl * vd)680ec727ea7Spatrick void TransferFunctions::reportConstRefUse(const Expr *ex, const VarDecl *vd) {
681ec727ea7Spatrick Value v = vals[vd];
682ec727ea7Spatrick if (isAlwaysUninit(v))
683ec727ea7Spatrick handler.handleConstRefUseOfUninitVariable(vd, getUninitUse(ex, vd, v));
684ec727ea7Spatrick }
685ec727ea7Spatrick
VisitObjCForCollectionStmt(ObjCForCollectionStmt * FS)686e5dd7070Spatrick void TransferFunctions::VisitObjCForCollectionStmt(ObjCForCollectionStmt *FS) {
687e5dd7070Spatrick // This represents an initialization of the 'element' value.
688e5dd7070Spatrick if (const auto *DS = dyn_cast<DeclStmt>(FS->getElement())) {
689e5dd7070Spatrick const auto *VD = cast<VarDecl>(DS->getSingleDecl());
690e5dd7070Spatrick if (isTrackedVar(VD))
691e5dd7070Spatrick vals[VD] = Initialized;
692e5dd7070Spatrick }
693e5dd7070Spatrick }
694e5dd7070Spatrick
VisitOMPExecutableDirective(OMPExecutableDirective * ED)695e5dd7070Spatrick void TransferFunctions::VisitOMPExecutableDirective(
696e5dd7070Spatrick OMPExecutableDirective *ED) {
697e5dd7070Spatrick for (Stmt *S : OMPExecutableDirective::used_clauses_children(ED->clauses())) {
698e5dd7070Spatrick assert(S && "Expected non-null used-in-clause child.");
699e5dd7070Spatrick Visit(S);
700e5dd7070Spatrick }
701e5dd7070Spatrick if (!ED->isStandaloneDirective())
702e5dd7070Spatrick Visit(ED->getStructuredBlock());
703e5dd7070Spatrick }
704e5dd7070Spatrick
VisitBlockExpr(BlockExpr * be)705e5dd7070Spatrick void TransferFunctions::VisitBlockExpr(BlockExpr *be) {
706e5dd7070Spatrick const BlockDecl *bd = be->getBlockDecl();
707e5dd7070Spatrick for (const auto &I : bd->captures()) {
708e5dd7070Spatrick const VarDecl *vd = I.getVariable();
709e5dd7070Spatrick if (!isTrackedVar(vd))
710e5dd7070Spatrick continue;
711e5dd7070Spatrick if (I.isByRef()) {
712e5dd7070Spatrick vals[vd] = Initialized;
713e5dd7070Spatrick continue;
714e5dd7070Spatrick }
715e5dd7070Spatrick reportUse(be, vd);
716e5dd7070Spatrick }
717e5dd7070Spatrick }
718e5dd7070Spatrick
VisitCallExpr(CallExpr * ce)719e5dd7070Spatrick void TransferFunctions::VisitCallExpr(CallExpr *ce) {
720e5dd7070Spatrick if (Decl *Callee = ce->getCalleeDecl()) {
721e5dd7070Spatrick if (Callee->hasAttr<ReturnsTwiceAttr>()) {
722e5dd7070Spatrick // After a call to a function like setjmp or vfork, any variable which is
723e5dd7070Spatrick // initialized anywhere within this function may now be initialized. For
724e5dd7070Spatrick // now, just assume such a call initializes all variables. FIXME: Only
725e5dd7070Spatrick // mark variables as initialized if they have an initializer which is
726e5dd7070Spatrick // reachable from here.
727e5dd7070Spatrick vals.setAllScratchValues(Initialized);
728e5dd7070Spatrick }
729e5dd7070Spatrick else if (Callee->hasAttr<AnalyzerNoReturnAttr>()) {
730e5dd7070Spatrick // Functions labeled like "analyzer_noreturn" are often used to denote
731e5dd7070Spatrick // "panic" functions that in special debug situations can still return,
732e5dd7070Spatrick // but for the most part should not be treated as returning. This is a
733e5dd7070Spatrick // useful annotation borrowed from the static analyzer that is useful for
734e5dd7070Spatrick // suppressing branch-specific false positives when we call one of these
735e5dd7070Spatrick // functions but keep pretending the path continues (when in reality the
736e5dd7070Spatrick // user doesn't care).
737e5dd7070Spatrick vals.setAllScratchValues(Unknown);
738e5dd7070Spatrick }
739e5dd7070Spatrick }
740e5dd7070Spatrick }
741e5dd7070Spatrick
VisitDeclRefExpr(DeclRefExpr * dr)742e5dd7070Spatrick void TransferFunctions::VisitDeclRefExpr(DeclRefExpr *dr) {
743e5dd7070Spatrick switch (classification.get(dr)) {
744e5dd7070Spatrick case ClassifyRefs::Ignore:
745e5dd7070Spatrick break;
746e5dd7070Spatrick case ClassifyRefs::Use:
747e5dd7070Spatrick reportUse(dr, cast<VarDecl>(dr->getDecl()));
748e5dd7070Spatrick break;
749e5dd7070Spatrick case ClassifyRefs::Init:
750e5dd7070Spatrick vals[cast<VarDecl>(dr->getDecl())] = Initialized;
751e5dd7070Spatrick break;
752e5dd7070Spatrick case ClassifyRefs::SelfInit:
753e5dd7070Spatrick handler.handleSelfInit(cast<VarDecl>(dr->getDecl()));
754e5dd7070Spatrick break;
755ec727ea7Spatrick case ClassifyRefs::ConstRefUse:
756ec727ea7Spatrick reportConstRefUse(dr, cast<VarDecl>(dr->getDecl()));
757ec727ea7Spatrick break;
758e5dd7070Spatrick }
759e5dd7070Spatrick }
760e5dd7070Spatrick
VisitBinaryOperator(BinaryOperator * BO)761e5dd7070Spatrick void TransferFunctions::VisitBinaryOperator(BinaryOperator *BO) {
762e5dd7070Spatrick if (BO->getOpcode() == BO_Assign) {
763e5dd7070Spatrick FindVarResult Var = findVar(BO->getLHS());
764e5dd7070Spatrick if (const VarDecl *VD = Var.getDecl())
765e5dd7070Spatrick vals[VD] = Initialized;
766e5dd7070Spatrick }
767e5dd7070Spatrick }
768e5dd7070Spatrick
VisitDeclStmt(DeclStmt * DS)769e5dd7070Spatrick void TransferFunctions::VisitDeclStmt(DeclStmt *DS) {
770e5dd7070Spatrick for (auto *DI : DS->decls()) {
771e5dd7070Spatrick auto *VD = dyn_cast<VarDecl>(DI);
772e5dd7070Spatrick if (VD && isTrackedVar(VD)) {
773e5dd7070Spatrick if (getSelfInitExpr(VD)) {
774e5dd7070Spatrick // If the initializer consists solely of a reference to itself, we
775e5dd7070Spatrick // explicitly mark the variable as uninitialized. This allows code
776e5dd7070Spatrick // like the following:
777e5dd7070Spatrick //
778e5dd7070Spatrick // int x = x;
779e5dd7070Spatrick //
780e5dd7070Spatrick // to deliberately leave a variable uninitialized. Different analysis
781e5dd7070Spatrick // clients can detect this pattern and adjust their reporting
782e5dd7070Spatrick // appropriately, but we need to continue to analyze subsequent uses
783e5dd7070Spatrick // of the variable.
784e5dd7070Spatrick vals[VD] = Uninitialized;
785e5dd7070Spatrick } else if (VD->getInit()) {
786e5dd7070Spatrick // Treat the new variable as initialized.
787e5dd7070Spatrick vals[VD] = Initialized;
788e5dd7070Spatrick } else {
789e5dd7070Spatrick // No initializer: the variable is now uninitialized. This matters
790e5dd7070Spatrick // for cases like:
791e5dd7070Spatrick // while (...) {
792e5dd7070Spatrick // int n;
793e5dd7070Spatrick // use(n);
794e5dd7070Spatrick // n = 0;
795e5dd7070Spatrick // }
796e5dd7070Spatrick // FIXME: Mark the variable as uninitialized whenever its scope is
797e5dd7070Spatrick // left, since its scope could be re-entered by a jump over the
798e5dd7070Spatrick // declaration.
799e5dd7070Spatrick vals[VD] = Uninitialized;
800e5dd7070Spatrick }
801e5dd7070Spatrick }
802e5dd7070Spatrick }
803e5dd7070Spatrick }
804e5dd7070Spatrick
VisitGCCAsmStmt(GCCAsmStmt * as)805ec727ea7Spatrick void TransferFunctions::VisitGCCAsmStmt(GCCAsmStmt *as) {
806ec727ea7Spatrick // An "asm goto" statement is a terminator that may initialize some variables.
807ec727ea7Spatrick if (!as->isAsmGoto())
808ec727ea7Spatrick return;
809ec727ea7Spatrick
810*12c85518Srobert ASTContext &C = ac.getASTContext();
811*12c85518Srobert for (const Expr *O : as->outputs()) {
812*12c85518Srobert const Expr *Ex = stripCasts(C, O);
813*12c85518Srobert
814*12c85518Srobert // Strip away any unary operators. Invalid l-values are reported by other
815*12c85518Srobert // semantic analysis passes.
816*12c85518Srobert while (const auto *UO = dyn_cast<UnaryOperator>(Ex))
817*12c85518Srobert Ex = stripCasts(C, UO->getSubExpr());
818*12c85518Srobert
819*12c85518Srobert // Mark the variable as potentially uninitialized for those cases where
820*12c85518Srobert // it's used on an indirect path, where it's not guaranteed to be
821*12c85518Srobert // defined.
822*12c85518Srobert if (const VarDecl *VD = findVar(Ex).getDecl())
823ec727ea7Spatrick vals[VD] = MayUninitialized;
824ec727ea7Spatrick }
825*12c85518Srobert }
826ec727ea7Spatrick
VisitObjCMessageExpr(ObjCMessageExpr * ME)827e5dd7070Spatrick void TransferFunctions::VisitObjCMessageExpr(ObjCMessageExpr *ME) {
828e5dd7070Spatrick // If the Objective-C message expression is an implicit no-return that
829e5dd7070Spatrick // is not modeled in the CFG, set the tracked dataflow values to Unknown.
830e5dd7070Spatrick if (objCNoRet.isImplicitNoReturn(ME)) {
831e5dd7070Spatrick vals.setAllScratchValues(Unknown);
832e5dd7070Spatrick }
833e5dd7070Spatrick }
834e5dd7070Spatrick
835e5dd7070Spatrick //------------------------------------------------------------------------====//
836e5dd7070Spatrick // High-level "driver" logic for uninitialized values analysis.
837e5dd7070Spatrick //====------------------------------------------------------------------------//
838e5dd7070Spatrick
runOnBlock(const CFGBlock * block,const CFG & cfg,AnalysisDeclContext & ac,CFGBlockValues & vals,const ClassifyRefs & classification,llvm::BitVector & wasAnalyzed,UninitVariablesHandler & handler)839e5dd7070Spatrick static bool runOnBlock(const CFGBlock *block, const CFG &cfg,
840e5dd7070Spatrick AnalysisDeclContext &ac, CFGBlockValues &vals,
841e5dd7070Spatrick const ClassifyRefs &classification,
842e5dd7070Spatrick llvm::BitVector &wasAnalyzed,
843e5dd7070Spatrick UninitVariablesHandler &handler) {
844e5dd7070Spatrick wasAnalyzed[block->getBlockID()] = true;
845e5dd7070Spatrick vals.resetScratch();
846e5dd7070Spatrick // Merge in values of predecessor blocks.
847e5dd7070Spatrick bool isFirst = true;
848e5dd7070Spatrick for (CFGBlock::const_pred_iterator I = block->pred_begin(),
849e5dd7070Spatrick E = block->pred_end(); I != E; ++I) {
850e5dd7070Spatrick const CFGBlock *pred = *I;
851e5dd7070Spatrick if (!pred)
852e5dd7070Spatrick continue;
853e5dd7070Spatrick if (wasAnalyzed[pred->getBlockID()]) {
854e5dd7070Spatrick vals.mergeIntoScratch(vals.getValueVector(pred), isFirst);
855e5dd7070Spatrick isFirst = false;
856e5dd7070Spatrick }
857e5dd7070Spatrick }
858e5dd7070Spatrick // Apply the transfer function.
859e5dd7070Spatrick TransferFunctions tf(vals, cfg, block, ac, classification, handler);
860e5dd7070Spatrick for (const auto &I : *block) {
861*12c85518Srobert if (std::optional<CFGStmt> cs = I.getAs<CFGStmt>())
862e5dd7070Spatrick tf.Visit(const_cast<Stmt *>(cs->getStmt()));
863e5dd7070Spatrick }
864ec727ea7Spatrick CFGTerminator terminator = block->getTerminator();
865ec727ea7Spatrick if (auto *as = dyn_cast_or_null<GCCAsmStmt>(terminator.getStmt()))
866ec727ea7Spatrick if (as->isAsmGoto())
867ec727ea7Spatrick tf.Visit(as);
868e5dd7070Spatrick return vals.updateValueVectorWithScratch(block);
869e5dd7070Spatrick }
870e5dd7070Spatrick
871e5dd7070Spatrick namespace {
872e5dd7070Spatrick
873e5dd7070Spatrick /// PruneBlocksHandler is a special UninitVariablesHandler that is used
874e5dd7070Spatrick /// to detect when a CFGBlock has any *potential* use of an uninitialized
875e5dd7070Spatrick /// variable. It is mainly used to prune out work during the final
876e5dd7070Spatrick /// reporting pass.
877e5dd7070Spatrick struct PruneBlocksHandler : public UninitVariablesHandler {
878e5dd7070Spatrick /// Records if a CFGBlock had a potential use of an uninitialized variable.
879e5dd7070Spatrick llvm::BitVector hadUse;
880e5dd7070Spatrick
881e5dd7070Spatrick /// Records if any CFGBlock had a potential use of an uninitialized variable.
882e5dd7070Spatrick bool hadAnyUse = false;
883e5dd7070Spatrick
884e5dd7070Spatrick /// The current block to scribble use information.
885e5dd7070Spatrick unsigned currentBlock = 0;
886e5dd7070Spatrick
PruneBlocksHandler__anonb5d9aead0811::PruneBlocksHandler887e5dd7070Spatrick PruneBlocksHandler(unsigned numBlocks) : hadUse(numBlocks, false) {}
888e5dd7070Spatrick
889e5dd7070Spatrick ~PruneBlocksHandler() override = default;
890e5dd7070Spatrick
handleUseOfUninitVariable__anonb5d9aead0811::PruneBlocksHandler891e5dd7070Spatrick void handleUseOfUninitVariable(const VarDecl *vd,
892e5dd7070Spatrick const UninitUse &use) override {
893e5dd7070Spatrick hadUse[currentBlock] = true;
894e5dd7070Spatrick hadAnyUse = true;
895e5dd7070Spatrick }
896e5dd7070Spatrick
handleConstRefUseOfUninitVariable__anonb5d9aead0811::PruneBlocksHandler897ec727ea7Spatrick void handleConstRefUseOfUninitVariable(const VarDecl *vd,
898ec727ea7Spatrick const UninitUse &use) override {
899ec727ea7Spatrick hadUse[currentBlock] = true;
900ec727ea7Spatrick hadAnyUse = true;
901ec727ea7Spatrick }
902ec727ea7Spatrick
903e5dd7070Spatrick /// Called when the uninitialized variable analysis detects the
904e5dd7070Spatrick /// idiom 'int x = x'. All other uses of 'x' within the initializer
905e5dd7070Spatrick /// are handled by handleUseOfUninitVariable.
handleSelfInit__anonb5d9aead0811::PruneBlocksHandler906e5dd7070Spatrick void handleSelfInit(const VarDecl *vd) override {
907e5dd7070Spatrick hadUse[currentBlock] = true;
908e5dd7070Spatrick hadAnyUse = true;
909e5dd7070Spatrick }
910e5dd7070Spatrick };
911e5dd7070Spatrick
912e5dd7070Spatrick } // namespace
913e5dd7070Spatrick
runUninitializedVariablesAnalysis(const DeclContext & dc,const CFG & cfg,AnalysisDeclContext & ac,UninitVariablesHandler & handler,UninitVariablesAnalysisStats & stats)914e5dd7070Spatrick void clang::runUninitializedVariablesAnalysis(
915e5dd7070Spatrick const DeclContext &dc,
916e5dd7070Spatrick const CFG &cfg,
917e5dd7070Spatrick AnalysisDeclContext &ac,
918e5dd7070Spatrick UninitVariablesHandler &handler,
919e5dd7070Spatrick UninitVariablesAnalysisStats &stats) {
920e5dd7070Spatrick CFGBlockValues vals(cfg);
921e5dd7070Spatrick vals.computeSetOfDeclarations(dc);
922e5dd7070Spatrick if (vals.hasNoDeclarations())
923e5dd7070Spatrick return;
924e5dd7070Spatrick
925e5dd7070Spatrick stats.NumVariablesAnalyzed = vals.getNumEntries();
926e5dd7070Spatrick
927e5dd7070Spatrick // Precompute which expressions are uses and which are initializations.
928e5dd7070Spatrick ClassifyRefs classification(ac);
929e5dd7070Spatrick cfg.VisitBlockStmts(classification);
930e5dd7070Spatrick
931e5dd7070Spatrick // Mark all variables uninitialized at the entry.
932e5dd7070Spatrick const CFGBlock &entry = cfg.getEntry();
933e5dd7070Spatrick ValueVector &vec = vals.getValueVector(&entry);
934e5dd7070Spatrick const unsigned n = vals.getNumEntries();
935e5dd7070Spatrick for (unsigned j = 0; j < n; ++j) {
936e5dd7070Spatrick vec[j] = Uninitialized;
937e5dd7070Spatrick }
938e5dd7070Spatrick
939e5dd7070Spatrick // Proceed with the workist.
940ec727ea7Spatrick ForwardDataflowWorklist worklist(cfg, ac);
941e5dd7070Spatrick llvm::BitVector previouslyVisited(cfg.getNumBlockIDs());
942e5dd7070Spatrick worklist.enqueueSuccessors(&cfg.getEntry());
943e5dd7070Spatrick llvm::BitVector wasAnalyzed(cfg.getNumBlockIDs(), false);
944e5dd7070Spatrick wasAnalyzed[cfg.getEntry().getBlockID()] = true;
945e5dd7070Spatrick PruneBlocksHandler PBH(cfg.getNumBlockIDs());
946e5dd7070Spatrick
947e5dd7070Spatrick while (const CFGBlock *block = worklist.dequeue()) {
948e5dd7070Spatrick PBH.currentBlock = block->getBlockID();
949e5dd7070Spatrick
950e5dd7070Spatrick // Did the block change?
951e5dd7070Spatrick bool changed = runOnBlock(block, cfg, ac, vals,
952e5dd7070Spatrick classification, wasAnalyzed, PBH);
953e5dd7070Spatrick ++stats.NumBlockVisits;
954e5dd7070Spatrick if (changed || !previouslyVisited[block->getBlockID()])
955e5dd7070Spatrick worklist.enqueueSuccessors(block);
956e5dd7070Spatrick previouslyVisited[block->getBlockID()] = true;
957e5dd7070Spatrick }
958e5dd7070Spatrick
959e5dd7070Spatrick if (!PBH.hadAnyUse)
960e5dd7070Spatrick return;
961e5dd7070Spatrick
962e5dd7070Spatrick // Run through the blocks one more time, and report uninitialized variables.
963e5dd7070Spatrick for (const auto *block : cfg)
964e5dd7070Spatrick if (PBH.hadUse[block->getBlockID()]) {
965e5dd7070Spatrick runOnBlock(block, cfg, ac, vals, classification, wasAnalyzed, handler);
966e5dd7070Spatrick ++stats.NumBlockVisits;
967e5dd7070Spatrick }
968e5dd7070Spatrick }
969e5dd7070Spatrick
970e5dd7070Spatrick UninitVariablesHandler::~UninitVariablesHandler() = default;
971