blob: 58191ec0b042ead5d8f406258f776f2d2e0c3f90 [file] [log] [blame]
Ted Kremenek6f342132011-03-15 03:17:07 +00001//==- UninitializedValues.cpp - Find Uninitialized Values -------*- C++ --*-==//
Ted Kremenek610068c2011-01-15 02:58:47 +00002//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// This file implements uninitialized values analysis for source-level CFGs.
11//
12//===----------------------------------------------------------------------===//
13
Ted Kremenek13bd4232011-01-20 17:37:17 +000014#include <utility>
Ted Kremenek610068c2011-01-15 02:58:47 +000015#include "llvm/ADT/Optional.h"
16#include "llvm/ADT/SmallVector.h"
17#include "llvm/ADT/BitVector.h"
18#include "llvm/ADT/DenseMap.h"
19#include "clang/AST/Decl.h"
20#include "clang/Analysis/CFG.h"
Ted Kremeneka8c17a52011-01-25 19:13:48 +000021#include "clang/Analysis/AnalysisContext.h"
Ted Kremenek610068c2011-01-15 02:58:47 +000022#include "clang/Analysis/Visitors/CFGRecStmtDeclVisitor.h"
Ted Kremenek6f342132011-03-15 03:17:07 +000023#include "clang/Analysis/Analyses/UninitializedValues.h"
Ted Kremenekc21fed32011-01-18 21:18:58 +000024#include "clang/Analysis/Support/SaveAndRestore.h"
Ted Kremenek610068c2011-01-15 02:58:47 +000025
26using namespace clang;
27
Ted Kremenek40900ee2011-01-27 02:29:34 +000028static bool isTrackedVar(const VarDecl *vd, const DeclContext *dc) {
Ted Kremenek1cbc3152011-03-17 03:06:11 +000029 if (vd->isLocalVarDecl() && !vd->hasGlobalStorage() &&
30 vd->getDeclContext() == dc) {
31 QualType ty = vd->getType();
32 return ty->isScalarType() || ty->isVectorType();
33 }
34 return false;
Ted Kremenekc104e532011-01-18 04:53:25 +000035}
36
Ted Kremenek610068c2011-01-15 02:58:47 +000037//------------------------------------------------------------------------====//
Ted Kremenek136f8f22011-03-15 04:57:27 +000038// DeclToIndex: a mapping from Decls we track to value indices.
Ted Kremenek610068c2011-01-15 02:58:47 +000039//====------------------------------------------------------------------------//
40
41namespace {
Ted Kremenek136f8f22011-03-15 04:57:27 +000042class DeclToIndex {
Ted Kremenek610068c2011-01-15 02:58:47 +000043 llvm::DenseMap<const VarDecl *, unsigned> map;
44public:
Ted Kremenek136f8f22011-03-15 04:57:27 +000045 DeclToIndex() {}
Ted Kremenek610068c2011-01-15 02:58:47 +000046
47 /// Compute the actual mapping from declarations to bits.
48 void computeMap(const DeclContext &dc);
49
50 /// Return the number of declarations in the map.
51 unsigned size() const { return map.size(); }
52
53 /// Returns the bit vector index for a given declaration.
Ted Kremenekb831c672011-03-29 01:40:00 +000054 llvm::Optional<unsigned> getValueIndex(const VarDecl *d) const;
Ted Kremenek610068c2011-01-15 02:58:47 +000055};
56}
57
Ted Kremenek136f8f22011-03-15 04:57:27 +000058void DeclToIndex::computeMap(const DeclContext &dc) {
Ted Kremenek610068c2011-01-15 02:58:47 +000059 unsigned count = 0;
60 DeclContext::specific_decl_iterator<VarDecl> I(dc.decls_begin()),
61 E(dc.decls_end());
62 for ( ; I != E; ++I) {
63 const VarDecl *vd = *I;
Ted Kremenek40900ee2011-01-27 02:29:34 +000064 if (isTrackedVar(vd, &dc))
Ted Kremenek610068c2011-01-15 02:58:47 +000065 map[vd] = count++;
66 }
67}
68
Ted Kremenekb831c672011-03-29 01:40:00 +000069llvm::Optional<unsigned> DeclToIndex::getValueIndex(const VarDecl *d) const {
70 llvm::DenseMap<const VarDecl *, unsigned>::const_iterator I = map.find(d);
Ted Kremenek610068c2011-01-15 02:58:47 +000071 if (I == map.end())
72 return llvm::Optional<unsigned>();
73 return I->second;
74}
75
76//------------------------------------------------------------------------====//
77// CFGBlockValues: dataflow values for CFG blocks.
78//====------------------------------------------------------------------------//
79
Ted Kremenekf7bafc72011-03-15 04:57:38 +000080// These values are defined in such a way that a merge can be done using
81// a bitwise OR.
82enum Value { Unknown = 0x0, /* 00 */
83 Initialized = 0x1, /* 01 */
84 Uninitialized = 0x2, /* 10 */
85 MayUninitialized = 0x3 /* 11 */ };
86
87static bool isUninitialized(const Value v) {
88 return v >= Uninitialized;
89}
90static bool isAlwaysUninit(const Value v) {
91 return v == Uninitialized;
92}
Ted Kremenekafb10c42011-03-15 04:57:29 +000093
Benjamin Kramerda57f3e2011-03-26 12:38:21 +000094namespace {
Ted Kremenekafb10c42011-03-15 04:57:29 +000095class ValueVector {
96 llvm::BitVector vec;
97public:
98 ValueVector() {}
Ted Kremenekf7bafc72011-03-15 04:57:38 +000099 ValueVector(unsigned size) : vec(size << 1) {}
100 void resize(unsigned n) { vec.resize(n << 1); }
Ted Kremenekafb10c42011-03-15 04:57:29 +0000101 void merge(const ValueVector &rhs) { vec |= rhs.vec; }
102 bool operator!=(const ValueVector &rhs) const { return vec != rhs.vec; }
Ted Kremenekafb10c42011-03-15 04:57:29 +0000103 void reset() { vec.reset(); }
Ted Kremenek496398d2011-03-15 04:57:32 +0000104
105 class reference {
106 ValueVector &vv;
107 const unsigned idx;
108
109 reference(); // Undefined
110 public:
111 reference(ValueVector &vv, unsigned idx) : vv(vv), idx(idx) {}
112 ~reference() {}
113
114 reference &operator=(Value v) {
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000115 vv.vec[idx << 1] = (((unsigned) v) & 0x1) ? true : false;
116 vv.vec[(idx << 1) | 1] = (((unsigned) v) & 0x2) ? true : false;
Ted Kremenek496398d2011-03-15 04:57:32 +0000117 return *this;
118 }
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000119 operator Value() {
120 unsigned x = (vv.vec[idx << 1] ? 1 : 0) | (vv.vec[(idx << 1) | 1] ? 2 :0);
121 return (Value) x;
122 }
Ted Kremenek496398d2011-03-15 04:57:32 +0000123 };
124
125 reference operator[](unsigned idx) { return reference(*this, idx); }
Ted Kremenekafb10c42011-03-15 04:57:29 +0000126};
127
Ted Kremenek136f8f22011-03-15 04:57:27 +0000128typedef std::pair<ValueVector *, ValueVector *> BVPair;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000129
Ted Kremenek610068c2011-01-15 02:58:47 +0000130class CFGBlockValues {
131 const CFG &cfg;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000132 BVPair *vals;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000133 ValueVector scratch;
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000134 DeclToIndex declToIndex;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000135
Ted Kremenek136f8f22011-03-15 04:57:27 +0000136 ValueVector &lazyCreate(ValueVector *&bv);
Ted Kremenek610068c2011-01-15 02:58:47 +0000137public:
138 CFGBlockValues(const CFG &cfg);
139 ~CFGBlockValues();
140
141 void computeSetOfDeclarations(const DeclContext &dc);
Ted Kremenek136f8f22011-03-15 04:57:27 +0000142 ValueVector &getValueVector(const CFGBlock *block,
Ted Kremenek13bd4232011-01-20 17:37:17 +0000143 const CFGBlock *dstBlock);
144
Ted Kremenek136f8f22011-03-15 04:57:27 +0000145 BVPair &getValueVectors(const CFGBlock *block, bool shouldLazyCreate);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000146
Ted Kremenek136f8f22011-03-15 04:57:27 +0000147 void mergeIntoScratch(ValueVector const &source, bool isFirst);
148 bool updateValueVectorWithScratch(const CFGBlock *block);
149 bool updateValueVectors(const CFGBlock *block, const BVPair &newVals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000150
151 bool hasNoDeclarations() const {
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000152 return declToIndex.size() == 0;
Ted Kremenek610068c2011-01-15 02:58:47 +0000153 }
154
Ted Kremenekb831c672011-03-29 01:40:00 +0000155 bool hasEntry(const VarDecl *vd) const {
156 return declToIndex.getValueIndex(vd).hasValue();
157 }
158
Ted Kremenek610068c2011-01-15 02:58:47 +0000159 void resetScratch();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000160 ValueVector &getScratch() { return scratch; }
Ted Kremenek13bd4232011-01-20 17:37:17 +0000161
Ted Kremenek136f8f22011-03-15 04:57:27 +0000162 ValueVector::reference operator[](const VarDecl *vd);
Ted Kremenek610068c2011-01-15 02:58:47 +0000163};
Benjamin Kramerda57f3e2011-03-26 12:38:21 +0000164} // end anonymous namespace
Ted Kremenek610068c2011-01-15 02:58:47 +0000165
166CFGBlockValues::CFGBlockValues(const CFG &c) : cfg(c), vals(0) {
167 unsigned n = cfg.getNumBlockIDs();
168 if (!n)
169 return;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000170 vals = new std::pair<ValueVector*, ValueVector*>[n];
Francois Pichet2d78c372011-01-15 13:27:47 +0000171 memset(vals, 0, sizeof(*vals) * n);
Ted Kremenek610068c2011-01-15 02:58:47 +0000172}
173
174CFGBlockValues::~CFGBlockValues() {
175 unsigned n = cfg.getNumBlockIDs();
176 if (n == 0)
177 return;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000178 for (unsigned i = 0; i < n; ++i) {
179 delete vals[i].first;
180 delete vals[i].second;
181 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000182 delete [] vals;
183}
184
185void CFGBlockValues::computeSetOfDeclarations(const DeclContext &dc) {
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000186 declToIndex.computeMap(dc);
187 scratch.resize(declToIndex.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000188}
189
Ted Kremenek136f8f22011-03-15 04:57:27 +0000190ValueVector &CFGBlockValues::lazyCreate(ValueVector *&bv) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000191 if (!bv)
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000192 bv = new ValueVector(declToIndex.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000193 return *bv;
194}
195
Ted Kremenek13bd4232011-01-20 17:37:17 +0000196/// This function pattern matches for a '&&' or '||' that appears at
197/// the beginning of a CFGBlock that also (1) has a terminator and
198/// (2) has no other elements. If such an expression is found, it is returned.
199static BinaryOperator *getLogicalOperatorInChain(const CFGBlock *block) {
200 if (block->empty())
201 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000202
Ted Kremenek3c0349e2011-03-01 03:15:10 +0000203 const CFGStmt *cstmt = block->front().getAs<CFGStmt>();
Ted Kremenek76709bf2011-03-15 05:22:28 +0000204 if (!cstmt)
205 return 0;
206
Ted Kremenek3c0349e2011-03-01 03:15:10 +0000207 BinaryOperator *b = llvm::dyn_cast_or_null<BinaryOperator>(cstmt->getStmt());
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000208
209 if (!b || !b->isLogicalOp())
Ted Kremenek13bd4232011-01-20 17:37:17 +0000210 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000211
212 if (block->pred_size() == 2 &&
213 ((block->succ_size() == 2 && block->getTerminatorCondition() == b) ||
214 block->size() == 1))
215 return b;
216
217 return 0;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000218}
219
Ted Kremenek136f8f22011-03-15 04:57:27 +0000220ValueVector &CFGBlockValues::getValueVector(const CFGBlock *block,
221 const CFGBlock *dstBlock) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000222 unsigned idx = block->getBlockID();
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000223 if (dstBlock && getLogicalOperatorInChain(block)) {
224 if (*block->succ_begin() == dstBlock)
225 return lazyCreate(vals[idx].first);
226 assert(*(block->succ_begin()+1) == dstBlock);
227 return lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000228 }
229
230 assert(vals[idx].second == 0);
231 return lazyCreate(vals[idx].first);
232}
233
Ted Kremenek136f8f22011-03-15 04:57:27 +0000234BVPair &CFGBlockValues::getValueVectors(const clang::CFGBlock *block,
235 bool shouldLazyCreate) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000236 unsigned idx = block->getBlockID();
237 lazyCreate(vals[idx].first);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000238 if (shouldLazyCreate)
239 lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000240 return vals[idx];
241}
242
Ted Kremenek136f8f22011-03-15 04:57:27 +0000243void CFGBlockValues::mergeIntoScratch(ValueVector const &source,
Ted Kremenek610068c2011-01-15 02:58:47 +0000244 bool isFirst) {
245 if (isFirst)
246 scratch = source;
247 else
Ted Kremenekafb10c42011-03-15 04:57:29 +0000248 scratch.merge(source);
Ted Kremenek610068c2011-01-15 02:58:47 +0000249}
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000250#if 0
Ted Kremenek136f8f22011-03-15 04:57:27 +0000251static void printVector(const CFGBlock *block, ValueVector &bv,
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000252 unsigned num) {
253
254 llvm::errs() << block->getBlockID() << " :";
255 for (unsigned i = 0; i < bv.size(); ++i) {
256 llvm::errs() << ' ' << bv[i];
257 }
258 llvm::errs() << " : " << num << '\n';
259}
260#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000261
Ted Kremenek136f8f22011-03-15 04:57:27 +0000262bool CFGBlockValues::updateValueVectorWithScratch(const CFGBlock *block) {
263 ValueVector &dst = getValueVector(block, 0);
Ted Kremenek610068c2011-01-15 02:58:47 +0000264 bool changed = (dst != scratch);
265 if (changed)
266 dst = scratch;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000267#if 0
268 printVector(block, scratch, 0);
269#endif
Ted Kremenek13bd4232011-01-20 17:37:17 +0000270 return changed;
271}
272
Ted Kremenek136f8f22011-03-15 04:57:27 +0000273bool CFGBlockValues::updateValueVectors(const CFGBlock *block,
Ted Kremenek13bd4232011-01-20 17:37:17 +0000274 const BVPair &newVals) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000275 BVPair &vals = getValueVectors(block, true);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000276 bool changed = *newVals.first != *vals.first ||
277 *newVals.second != *vals.second;
278 *vals.first = *newVals.first;
279 *vals.second = *newVals.second;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000280#if 0
281 printVector(block, *vals.first, 1);
282 printVector(block, *vals.second, 2);
283#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000284 return changed;
285}
286
287void CFGBlockValues::resetScratch() {
288 scratch.reset();
289}
290
Ted Kremenek136f8f22011-03-15 04:57:27 +0000291ValueVector::reference CFGBlockValues::operator[](const VarDecl *vd) {
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000292 const llvm::Optional<unsigned> &idx = declToIndex.getValueIndex(vd);
Ted Kremenek610068c2011-01-15 02:58:47 +0000293 assert(idx.hasValue());
294 return scratch[idx.getValue()];
295}
296
297//------------------------------------------------------------------------====//
298// Worklist: worklist for dataflow analysis.
299//====------------------------------------------------------------------------//
300
301namespace {
302class DataflowWorklist {
303 llvm::SmallVector<const CFGBlock *, 20> worklist;
Ted Kremenek496398d2011-03-15 04:57:32 +0000304 llvm::BitVector enqueuedBlocks;
Ted Kremenek610068c2011-01-15 02:58:47 +0000305public:
306 DataflowWorklist(const CFG &cfg) : enqueuedBlocks(cfg.getNumBlockIDs()) {}
307
308 void enqueue(const CFGBlock *block);
309 void enqueueSuccessors(const CFGBlock *block);
310 const CFGBlock *dequeue();
311
312};
313}
314
315void DataflowWorklist::enqueue(const CFGBlock *block) {
Ted Kremenekc104e532011-01-18 04:53:25 +0000316 if (!block)
317 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000318 unsigned idx = block->getBlockID();
319 if (enqueuedBlocks[idx])
320 return;
321 worklist.push_back(block);
322 enqueuedBlocks[idx] = true;
323}
324
325void DataflowWorklist::enqueueSuccessors(const clang::CFGBlock *block) {
326 for (CFGBlock::const_succ_iterator I = block->succ_begin(),
327 E = block->succ_end(); I != E; ++I) {
328 enqueue(*I);
329 }
330}
331
332const CFGBlock *DataflowWorklist::dequeue() {
333 if (worklist.empty())
334 return 0;
335 const CFGBlock *b = worklist.back();
336 worklist.pop_back();
337 enqueuedBlocks[b->getBlockID()] = false;
338 return b;
339}
340
341//------------------------------------------------------------------------====//
342// Transfer function for uninitialized values analysis.
343//====------------------------------------------------------------------------//
344
Ted Kremenek610068c2011-01-15 02:58:47 +0000345namespace {
346class FindVarResult {
347 const VarDecl *vd;
348 const DeclRefExpr *dr;
349public:
350 FindVarResult(VarDecl *vd, DeclRefExpr *dr) : vd(vd), dr(dr) {}
351
352 const DeclRefExpr *getDeclRefExpr() const { return dr; }
353 const VarDecl *getDecl() const { return vd; }
354};
355
356class TransferFunctions : public CFGRecStmtVisitor<TransferFunctions> {
357 CFGBlockValues &vals;
358 const CFG &cfg;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000359 AnalysisContext &ac;
Ted Kremenek610068c2011-01-15 02:58:47 +0000360 UninitVariablesHandler *handler;
Ted Kremenekc21fed32011-01-18 21:18:58 +0000361 const DeclRefExpr *currentDR;
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000362 const Expr *currentVoidCast;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000363 const bool flagBlockUses;
Ted Kremenek610068c2011-01-15 02:58:47 +0000364public:
365 TransferFunctions(CFGBlockValues &vals, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000366 AnalysisContext &ac,
367 UninitVariablesHandler *handler,
368 bool flagBlockUses)
369 : vals(vals), cfg(cfg), ac(ac), handler(handler), currentDR(0),
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000370 currentVoidCast(0), flagBlockUses(flagBlockUses) {}
Ted Kremenek610068c2011-01-15 02:58:47 +0000371
372 const CFG &getCFG() { return cfg; }
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000373 void reportUninit(const DeclRefExpr *ex, const VarDecl *vd,
374 bool isAlwaysUninit);
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000375
376 void VisitBlockExpr(BlockExpr *be);
Ted Kremenek610068c2011-01-15 02:58:47 +0000377 void VisitDeclStmt(DeclStmt *ds);
Ted Kremenekc21fed32011-01-18 21:18:58 +0000378 void VisitDeclRefExpr(DeclRefExpr *dr);
Ted Kremenek610068c2011-01-15 02:58:47 +0000379 void VisitUnaryOperator(UnaryOperator *uo);
380 void VisitBinaryOperator(BinaryOperator *bo);
381 void VisitCastExpr(CastExpr *ce);
Peter Collingbournef4e3cfb2011-03-11 19:24:49 +0000382 void VisitUnaryExprOrTypeTraitExpr(UnaryExprOrTypeTraitExpr *se);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000383 void BlockStmt_VisitObjCForCollectionStmt(ObjCForCollectionStmt *fs);
Ted Kremenek40900ee2011-01-27 02:29:34 +0000384
385 bool isTrackedVar(const VarDecl *vd) {
Ted Kremenekb831c672011-03-29 01:40:00 +0000386#if 1
387 // FIXME: This is a temporary workaround to deal with the fact
388 // that DeclContext's do not always contain all of their variables!
389 return vals.hasEntry(vd);
390#else
Ted Kremenek40900ee2011-01-27 02:29:34 +0000391 return ::isTrackedVar(vd, cast<DeclContext>(ac.getDecl()));
Ted Kremenekb831c672011-03-29 01:40:00 +0000392#endif
Ted Kremenek40900ee2011-01-27 02:29:34 +0000393 }
394
395 FindVarResult findBlockVarDecl(Expr *ex);
Ted Kremenek610068c2011-01-15 02:58:47 +0000396};
397}
398
399void TransferFunctions::reportUninit(const DeclRefExpr *ex,
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000400 const VarDecl *vd, bool isAlwaysUnit) {
401 if (handler) handler->handleUseOfUninitVariable(ex, vd, isAlwaysUnit);
Ted Kremenek610068c2011-01-15 02:58:47 +0000402}
403
Ted Kremenek40900ee2011-01-27 02:29:34 +0000404FindVarResult TransferFunctions::findBlockVarDecl(Expr* ex) {
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000405 if (DeclRefExpr* dr = dyn_cast<DeclRefExpr>(ex->IgnoreParenCasts()))
406 if (VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
407 if (isTrackedVar(vd))
Ted Kremenek40900ee2011-01-27 02:29:34 +0000408 return FindVarResult(vd, dr);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000409 return FindVarResult(0, 0);
410}
411
412void TransferFunctions::BlockStmt_VisitObjCForCollectionStmt(
413 ObjCForCollectionStmt *fs) {
414
415 Visit(fs->getCollection());
416
417 // This represents an initialization of the 'element' value.
418 Stmt *element = fs->getElement();
419 const VarDecl* vd = 0;
420
421 if (DeclStmt* ds = dyn_cast<DeclStmt>(element)) {
422 vd = cast<VarDecl>(ds->getSingleDecl());
423 if (!isTrackedVar(vd))
424 vd = 0;
425 }
426 else {
427 // Initialize the value of the reference variable.
428 const FindVarResult &res = findBlockVarDecl(cast<Expr>(element));
429 vd = res.getDecl();
430 if (!vd) {
431 Visit(element);
432 return;
433 }
434 }
435
436 if (vd)
437 vals[vd] = Initialized;
438}
439
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000440void TransferFunctions::VisitBlockExpr(BlockExpr *be) {
441 if (!flagBlockUses || !handler)
442 return;
Ted Kremenekbc8b44c2011-03-31 22:32:41 +0000443 const BlockDecl *bd = be->getBlockDecl();
444 for (BlockDecl::capture_const_iterator i = bd->capture_begin(),
445 e = bd->capture_end() ; i != e; ++i) {
446 const VarDecl *vd = i->getVariable();
447 if (!vd->hasLocalStorage())
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000448 continue;
Ted Kremenekbc8b44c2011-03-31 22:32:41 +0000449 if (!isTrackedVar(vd))
450 continue;
451 if (i->isByRef()) {
452 vals[vd] = Initialized;
453 continue;
454 }
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000455 Value v = vals[vd];
456 if (isUninitialized(v))
457 handler->handleUseOfUninitVariable(be, vd, isAlwaysUninit(v));
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000458 }
459}
460
Ted Kremenek610068c2011-01-15 02:58:47 +0000461void TransferFunctions::VisitDeclStmt(DeclStmt *ds) {
462 for (DeclStmt::decl_iterator DI = ds->decl_begin(), DE = ds->decl_end();
463 DI != DE; ++DI) {
464 if (VarDecl *vd = dyn_cast<VarDecl>(*DI)) {
Ted Kremenek4dccb902011-01-18 05:00:42 +0000465 if (isTrackedVar(vd)) {
466 vals[vd] = Uninitialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000467 if (Stmt *init = vd->getInit()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000468 Visit(init);
Ted Kremenekc104e532011-01-18 04:53:25 +0000469 vals[vd] = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000470 }
Ted Kremenek4dccb902011-01-18 05:00:42 +0000471 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000472 else if (Stmt *init = vd->getInit()) {
473 Visit(init);
474 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000475 }
476 }
477}
478
Ted Kremenekc21fed32011-01-18 21:18:58 +0000479void TransferFunctions::VisitDeclRefExpr(DeclRefExpr *dr) {
480 // We assume that DeclRefExprs wrapped in an lvalue-to-rvalue cast
481 // cannot be block-level expressions. Therefore, we determine if
482 // a DeclRefExpr is involved in a "load" by comparing it to the current
483 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
484 // If a DeclRefExpr is not involved in a load, we are essentially computing
485 // its address, either for assignment to a reference or via the '&' operator.
486 // In such cases, treat the variable as being initialized, since this
487 // analysis isn't powerful enough to do alias tracking.
488 if (dr != currentDR)
489 if (const VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
490 if (isTrackedVar(vd))
491 vals[vd] = Initialized;
492}
493
Ted Kremenek610068c2011-01-15 02:58:47 +0000494void TransferFunctions::VisitBinaryOperator(clang::BinaryOperator *bo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000495 if (bo->isAssignmentOp()) {
496 const FindVarResult &res = findBlockVarDecl(bo->getLHS());
497 if (const VarDecl* vd = res.getDecl()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000498 // We assume that DeclRefExprs wrapped in a BinaryOperator "assignment"
499 // cannot be block-level expressions. Therefore, we determine if
500 // a DeclRefExpr is involved in a "load" by comparing it to the current
501 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
502 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
503 res.getDeclRefExpr());
504 Visit(bo->getRHS());
505 Visit(bo->getLHS());
506
Ted Kremenek496398d2011-03-15 04:57:32 +0000507 ValueVector::reference val = vals[vd];
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000508 if (isUninitialized(val)) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000509 if (bo->getOpcode() != BO_Assign)
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000510 reportUninit(res.getDeclRefExpr(), vd, isAlwaysUninit(val));
Ted Kremenek496398d2011-03-15 04:57:32 +0000511 val = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000512 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000513 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000514 }
515 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000516 Visit(bo->getRHS());
517 Visit(bo->getLHS());
Ted Kremenek610068c2011-01-15 02:58:47 +0000518}
519
520void TransferFunctions::VisitUnaryOperator(clang::UnaryOperator *uo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000521 switch (uo->getOpcode()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000522 case clang::UO_PostDec:
523 case clang::UO_PostInc:
524 case clang::UO_PreDec:
525 case clang::UO_PreInc: {
526 const FindVarResult &res = findBlockVarDecl(uo->getSubExpr());
527 if (const VarDecl *vd = res.getDecl()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000528 // We assume that DeclRefExprs wrapped in a unary operator ++/--
529 // cannot be block-level expressions. Therefore, we determine if
530 // a DeclRefExpr is involved in a "load" by comparing it to the current
531 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
532 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
533 res.getDeclRefExpr());
534 Visit(uo->getSubExpr());
535
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000536 ValueVector::reference val = vals[vd];
537 if (isUninitialized(val)) {
538 reportUninit(res.getDeclRefExpr(), vd, isAlwaysUninit(val));
539 // Don't cascade warnings.
540 val = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000541 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000542 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000543 }
544 break;
545 }
546 default:
547 break;
548 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000549 Visit(uo->getSubExpr());
Ted Kremenek610068c2011-01-15 02:58:47 +0000550}
551
552void TransferFunctions::VisitCastExpr(clang::CastExpr *ce) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000553 if (ce->getCastKind() == CK_LValueToRValue) {
554 const FindVarResult &res = findBlockVarDecl(ce->getSubExpr());
Ted Kremenekc21fed32011-01-18 21:18:58 +0000555 if (const VarDecl *vd = res.getDecl()) {
556 // We assume that DeclRefExprs wrapped in an lvalue-to-rvalue cast
557 // cannot be block-level expressions. Therefore, we determine if
558 // a DeclRefExpr is involved in a "load" by comparing it to the current
559 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
560 // Here we update 'currentDR' to be the one associated with this
561 // lvalue-to-rvalue cast. Then, when we analyze the DeclRefExpr, we
562 // will know that we are not computing its lvalue for other purposes
563 // than to perform a load.
564 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
565 res.getDeclRefExpr());
566 Visit(ce->getSubExpr());
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000567 if (currentVoidCast != ce) {
568 Value val = vals[vd];
569 if (isUninitialized(val)) {
570 reportUninit(res.getDeclRefExpr(), vd, isAlwaysUninit(val));
571 // Don't cascade warnings.
572 vals[vd] = Initialized;
573 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000574 }
575 return;
576 }
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000577 }
578 else if (CStyleCastExpr *cse = dyn_cast<CStyleCastExpr>(ce)) {
579 if (cse->getType()->isVoidType()) {
580 // e.g. (void) x;
581 SaveAndRestore<const Expr *>
582 lastVoidCast(currentVoidCast, cse->getSubExpr()->IgnoreParens());
583 Visit(cse->getSubExpr());
584 return;
585 }
586 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000587 Visit(ce->getSubExpr());
Ted Kremenek610068c2011-01-15 02:58:47 +0000588}
589
Peter Collingbournef4e3cfb2011-03-11 19:24:49 +0000590void TransferFunctions::VisitUnaryExprOrTypeTraitExpr(
591 UnaryExprOrTypeTraitExpr *se) {
592 if (se->getKind() == UETT_SizeOf) {
Ted Kremenek96608032011-01-23 17:53:04 +0000593 if (se->getType()->isConstantSizeType())
594 return;
595 // Handle VLAs.
596 Visit(se->getArgumentExpr());
597 }
598}
599
Ted Kremenek610068c2011-01-15 02:58:47 +0000600//------------------------------------------------------------------------====//
601// High-level "driver" logic for uninitialized values analysis.
602//====------------------------------------------------------------------------//
603
Ted Kremenek13bd4232011-01-20 17:37:17 +0000604static bool runOnBlock(const CFGBlock *block, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000605 AnalysisContext &ac, CFGBlockValues &vals,
606 UninitVariablesHandler *handler = 0,
607 bool flagBlockUses = false) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000608
609 if (const BinaryOperator *b = getLogicalOperatorInChain(block)) {
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000610 CFGBlock::const_pred_iterator itr = block->pred_begin();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000611 BVPair vA = vals.getValueVectors(*itr, false);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000612 ++itr;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000613 BVPair vB = vals.getValueVectors(*itr, false);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000614
615 BVPair valsAB;
616
617 if (b->getOpcode() == BO_LAnd) {
618 // Merge the 'F' bits from the first and second.
619 vals.mergeIntoScratch(*(vA.second ? vA.second : vA.first), true);
620 vals.mergeIntoScratch(*(vB.second ? vB.second : vB.first), false);
621 valsAB.first = vA.first;
Ted Kremenek2d4bed12011-01-20 21:25:31 +0000622 valsAB.second = &vals.getScratch();
Ted Kremenek13bd4232011-01-20 17:37:17 +0000623 }
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000624 else {
625 // Merge the 'T' bits from the first and second.
626 assert(b->getOpcode() == BO_LOr);
627 vals.mergeIntoScratch(*vA.first, true);
628 vals.mergeIntoScratch(*vB.first, false);
629 valsAB.first = &vals.getScratch();
630 valsAB.second = vA.second ? vA.second : vA.first;
631 }
Ted Kremenek136f8f22011-03-15 04:57:27 +0000632 return vals.updateValueVectors(block, valsAB);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000633 }
634
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000635 // Default behavior: merge in values of predecessor blocks.
Ted Kremenek610068c2011-01-15 02:58:47 +0000636 vals.resetScratch();
637 bool isFirst = true;
638 for (CFGBlock::const_pred_iterator I = block->pred_begin(),
639 E = block->pred_end(); I != E; ++I) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000640 vals.mergeIntoScratch(vals.getValueVector(*I, block), isFirst);
Ted Kremenek610068c2011-01-15 02:58:47 +0000641 isFirst = false;
642 }
643 // Apply the transfer function.
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000644 TransferFunctions tf(vals, cfg, ac, handler, flagBlockUses);
Ted Kremenek610068c2011-01-15 02:58:47 +0000645 for (CFGBlock::const_iterator I = block->begin(), E = block->end();
646 I != E; ++I) {
647 if (const CFGStmt *cs = dyn_cast<CFGStmt>(&*I)) {
648 tf.BlockStmt_Visit(cs->getStmt());
649 }
650 }
Ted Kremenek136f8f22011-03-15 04:57:27 +0000651 return vals.updateValueVectorWithScratch(block);
Ted Kremenek610068c2011-01-15 02:58:47 +0000652}
653
654void clang::runUninitializedVariablesAnalysis(const DeclContext &dc,
655 const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000656 AnalysisContext &ac,
Ted Kremenek610068c2011-01-15 02:58:47 +0000657 UninitVariablesHandler &handler) {
658 CFGBlockValues vals(cfg);
659 vals.computeSetOfDeclarations(dc);
660 if (vals.hasNoDeclarations())
661 return;
662 DataflowWorklist worklist(cfg);
Ted Kremenek496398d2011-03-15 04:57:32 +0000663 llvm::BitVector previouslyVisited(cfg.getNumBlockIDs());
Ted Kremenek610068c2011-01-15 02:58:47 +0000664
665 worklist.enqueueSuccessors(&cfg.getEntry());
666
667 while (const CFGBlock *block = worklist.dequeue()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000668 // Did the block change?
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000669 bool changed = runOnBlock(block, cfg, ac, vals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000670 if (changed || !previouslyVisited[block->getBlockID()])
671 worklist.enqueueSuccessors(block);
672 previouslyVisited[block->getBlockID()] = true;
673 }
674
675 // Run through the blocks one more time, and report uninitialized variabes.
676 for (CFG::const_iterator BI = cfg.begin(), BE = cfg.end(); BI != BE; ++BI) {
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000677 runOnBlock(*BI, cfg, ac, vals, &handler, /* flagBlockUses */ true);
Ted Kremenek610068c2011-01-15 02:58:47 +0000678 }
679}
680
681UninitVariablesHandler::~UninitVariablesHandler() {}
682