blob: 31ee412815c241960b3c9bba5d7474568307c7e6 [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 Kremenekc104e532011-01-18 04:53:25 +000029 return vd->isLocalVarDecl() && !vd->hasGlobalStorage() &&
Ted Kremenek40900ee2011-01-27 02:29:34 +000030 vd->getType()->isScalarType() &&
31 vd->getDeclContext() == dc;
Ted Kremenekc104e532011-01-18 04:53:25 +000032}
33
Ted Kremenek610068c2011-01-15 02:58:47 +000034//------------------------------------------------------------------------====//
Ted Kremenek136f8f22011-03-15 04:57:27 +000035// DeclToIndex: a mapping from Decls we track to value indices.
Ted Kremenek610068c2011-01-15 02:58:47 +000036//====------------------------------------------------------------------------//
37
38namespace {
Ted Kremenek136f8f22011-03-15 04:57:27 +000039class DeclToIndex {
Ted Kremenek610068c2011-01-15 02:58:47 +000040 llvm::DenseMap<const VarDecl *, unsigned> map;
41public:
Ted Kremenek136f8f22011-03-15 04:57:27 +000042 DeclToIndex() {}
Ted Kremenek610068c2011-01-15 02:58:47 +000043
44 /// Compute the actual mapping from declarations to bits.
45 void computeMap(const DeclContext &dc);
46
47 /// Return the number of declarations in the map.
48 unsigned size() const { return map.size(); }
49
50 /// Returns the bit vector index for a given declaration.
Ted Kremenek136f8f22011-03-15 04:57:27 +000051 llvm::Optional<unsigned> getValueIndex(const VarDecl *d);
Ted Kremenek610068c2011-01-15 02:58:47 +000052};
53}
54
Ted Kremenek136f8f22011-03-15 04:57:27 +000055void DeclToIndex::computeMap(const DeclContext &dc) {
Ted Kremenek610068c2011-01-15 02:58:47 +000056 unsigned count = 0;
57 DeclContext::specific_decl_iterator<VarDecl> I(dc.decls_begin()),
58 E(dc.decls_end());
59 for ( ; I != E; ++I) {
60 const VarDecl *vd = *I;
Ted Kremenek40900ee2011-01-27 02:29:34 +000061 if (isTrackedVar(vd, &dc))
Ted Kremenek610068c2011-01-15 02:58:47 +000062 map[vd] = count++;
63 }
64}
65
Ted Kremenek136f8f22011-03-15 04:57:27 +000066llvm::Optional<unsigned> DeclToIndex::getValueIndex(const VarDecl *d) {
Ted Kremenek610068c2011-01-15 02:58:47 +000067 llvm::DenseMap<const VarDecl *, unsigned>::iterator I = map.find(d);
68 if (I == map.end())
69 return llvm::Optional<unsigned>();
70 return I->second;
71}
72
73//------------------------------------------------------------------------====//
74// CFGBlockValues: dataflow values for CFG blocks.
75//====------------------------------------------------------------------------//
76
Ted Kremenekf7bafc72011-03-15 04:57:38 +000077// These values are defined in such a way that a merge can be done using
78// a bitwise OR.
79enum Value { Unknown = 0x0, /* 00 */
80 Initialized = 0x1, /* 01 */
81 Uninitialized = 0x2, /* 10 */
82 MayUninitialized = 0x3 /* 11 */ };
83
84static bool isUninitialized(const Value v) {
85 return v >= Uninitialized;
86}
87static bool isAlwaysUninit(const Value v) {
88 return v == Uninitialized;
89}
Ted Kremenekafb10c42011-03-15 04:57:29 +000090
91class ValueVector {
92 llvm::BitVector vec;
93public:
94 ValueVector() {}
Ted Kremenekf7bafc72011-03-15 04:57:38 +000095 ValueVector(unsigned size) : vec(size << 1) {}
96 void resize(unsigned n) { vec.resize(n << 1); }
Ted Kremenekafb10c42011-03-15 04:57:29 +000097 void merge(const ValueVector &rhs) { vec |= rhs.vec; }
98 bool operator!=(const ValueVector &rhs) const { return vec != rhs.vec; }
Ted Kremenekafb10c42011-03-15 04:57:29 +000099 void reset() { vec.reset(); }
Ted Kremenek496398d2011-03-15 04:57:32 +0000100
101 class reference {
102 ValueVector &vv;
103 const unsigned idx;
104
105 reference(); // Undefined
106 public:
107 reference(ValueVector &vv, unsigned idx) : vv(vv), idx(idx) {}
108 ~reference() {}
109
110 reference &operator=(Value v) {
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000111 vv.vec[idx << 1] = (((unsigned) v) & 0x1) ? true : false;
112 vv.vec[(idx << 1) | 1] = (((unsigned) v) & 0x2) ? true : false;
Ted Kremenek496398d2011-03-15 04:57:32 +0000113 return *this;
114 }
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000115 operator Value() {
116 unsigned x = (vv.vec[idx << 1] ? 1 : 0) | (vv.vec[(idx << 1) | 1] ? 2 :0);
117 return (Value) x;
118 }
119
Ted Kremenek496398d2011-03-15 04:57:32 +0000120 bool operator==(Value v) {
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000121 return v = operator Value();
Ted Kremenek496398d2011-03-15 04:57:32 +0000122 }
123 };
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 +0000130namespace {
131class CFGBlockValues {
132 const CFG &cfg;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000133 BVPair *vals;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000134 ValueVector scratch;
135 DeclToIndex DeclToIndex;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000136
Ted Kremenek136f8f22011-03-15 04:57:27 +0000137 ValueVector &lazyCreate(ValueVector *&bv);
Ted Kremenek610068c2011-01-15 02:58:47 +0000138public:
139 CFGBlockValues(const CFG &cfg);
140 ~CFGBlockValues();
141
142 void computeSetOfDeclarations(const DeclContext &dc);
Ted Kremenek136f8f22011-03-15 04:57:27 +0000143 ValueVector &getValueVector(const CFGBlock *block,
Ted Kremenek13bd4232011-01-20 17:37:17 +0000144 const CFGBlock *dstBlock);
145
Ted Kremenek136f8f22011-03-15 04:57:27 +0000146 BVPair &getValueVectors(const CFGBlock *block, bool shouldLazyCreate);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000147
Ted Kremenek136f8f22011-03-15 04:57:27 +0000148 void mergeIntoScratch(ValueVector const &source, bool isFirst);
149 bool updateValueVectorWithScratch(const CFGBlock *block);
150 bool updateValueVectors(const CFGBlock *block, const BVPair &newVals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000151
152 bool hasNoDeclarations() const {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000153 return DeclToIndex.size() == 0;
Ted Kremenek610068c2011-01-15 02:58:47 +0000154 }
155
156 void resetScratch();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000157 ValueVector &getScratch() { return scratch; }
Ted Kremenek13bd4232011-01-20 17:37:17 +0000158
Ted Kremenek136f8f22011-03-15 04:57:27 +0000159 ValueVector::reference operator[](const VarDecl *vd);
Ted Kremenek610068c2011-01-15 02:58:47 +0000160};
161}
162
163CFGBlockValues::CFGBlockValues(const CFG &c) : cfg(c), vals(0) {
164 unsigned n = cfg.getNumBlockIDs();
165 if (!n)
166 return;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000167 vals = new std::pair<ValueVector*, ValueVector*>[n];
Francois Pichet2d78c372011-01-15 13:27:47 +0000168 memset(vals, 0, sizeof(*vals) * n);
Ted Kremenek610068c2011-01-15 02:58:47 +0000169}
170
171CFGBlockValues::~CFGBlockValues() {
172 unsigned n = cfg.getNumBlockIDs();
173 if (n == 0)
174 return;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000175 for (unsigned i = 0; i < n; ++i) {
176 delete vals[i].first;
177 delete vals[i].second;
178 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000179 delete [] vals;
180}
181
182void CFGBlockValues::computeSetOfDeclarations(const DeclContext &dc) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000183 DeclToIndex.computeMap(dc);
184 scratch.resize(DeclToIndex.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000185}
186
Ted Kremenek136f8f22011-03-15 04:57:27 +0000187ValueVector &CFGBlockValues::lazyCreate(ValueVector *&bv) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000188 if (!bv)
Ted Kremenek136f8f22011-03-15 04:57:27 +0000189 bv = new ValueVector(DeclToIndex.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000190 return *bv;
191}
192
Ted Kremenek13bd4232011-01-20 17:37:17 +0000193/// This function pattern matches for a '&&' or '||' that appears at
194/// the beginning of a CFGBlock that also (1) has a terminator and
195/// (2) has no other elements. If such an expression is found, it is returned.
196static BinaryOperator *getLogicalOperatorInChain(const CFGBlock *block) {
197 if (block->empty())
198 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000199
Ted Kremenek3c0349e2011-03-01 03:15:10 +0000200 const CFGStmt *cstmt = block->front().getAs<CFGStmt>();
Ted Kremenek76709bf2011-03-15 05:22:28 +0000201 if (!cstmt)
202 return 0;
203
Ted Kremenek3c0349e2011-03-01 03:15:10 +0000204 BinaryOperator *b = llvm::dyn_cast_or_null<BinaryOperator>(cstmt->getStmt());
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000205
206 if (!b || !b->isLogicalOp())
Ted Kremenek13bd4232011-01-20 17:37:17 +0000207 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000208
209 if (block->pred_size() == 2 &&
210 ((block->succ_size() == 2 && block->getTerminatorCondition() == b) ||
211 block->size() == 1))
212 return b;
213
214 return 0;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000215}
216
Ted Kremenek136f8f22011-03-15 04:57:27 +0000217ValueVector &CFGBlockValues::getValueVector(const CFGBlock *block,
218 const CFGBlock *dstBlock) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000219 unsigned idx = block->getBlockID();
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000220 if (dstBlock && getLogicalOperatorInChain(block)) {
221 if (*block->succ_begin() == dstBlock)
222 return lazyCreate(vals[idx].first);
223 assert(*(block->succ_begin()+1) == dstBlock);
224 return lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000225 }
226
227 assert(vals[idx].second == 0);
228 return lazyCreate(vals[idx].first);
229}
230
Ted Kremenek136f8f22011-03-15 04:57:27 +0000231BVPair &CFGBlockValues::getValueVectors(const clang::CFGBlock *block,
232 bool shouldLazyCreate) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000233 unsigned idx = block->getBlockID();
234 lazyCreate(vals[idx].first);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000235 if (shouldLazyCreate)
236 lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000237 return vals[idx];
238}
239
Ted Kremenek136f8f22011-03-15 04:57:27 +0000240void CFGBlockValues::mergeIntoScratch(ValueVector const &source,
Ted Kremenek610068c2011-01-15 02:58:47 +0000241 bool isFirst) {
242 if (isFirst)
243 scratch = source;
244 else
Ted Kremenekafb10c42011-03-15 04:57:29 +0000245 scratch.merge(source);
Ted Kremenek610068c2011-01-15 02:58:47 +0000246}
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000247#if 0
Ted Kremenek136f8f22011-03-15 04:57:27 +0000248static void printVector(const CFGBlock *block, ValueVector &bv,
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000249 unsigned num) {
250
251 llvm::errs() << block->getBlockID() << " :";
252 for (unsigned i = 0; i < bv.size(); ++i) {
253 llvm::errs() << ' ' << bv[i];
254 }
255 llvm::errs() << " : " << num << '\n';
256}
257#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000258
Ted Kremenek136f8f22011-03-15 04:57:27 +0000259bool CFGBlockValues::updateValueVectorWithScratch(const CFGBlock *block) {
260 ValueVector &dst = getValueVector(block, 0);
Ted Kremenek610068c2011-01-15 02:58:47 +0000261 bool changed = (dst != scratch);
262 if (changed)
263 dst = scratch;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000264#if 0
265 printVector(block, scratch, 0);
266#endif
Ted Kremenek13bd4232011-01-20 17:37:17 +0000267 return changed;
268}
269
Ted Kremenek136f8f22011-03-15 04:57:27 +0000270bool CFGBlockValues::updateValueVectors(const CFGBlock *block,
Ted Kremenek13bd4232011-01-20 17:37:17 +0000271 const BVPair &newVals) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000272 BVPair &vals = getValueVectors(block, true);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000273 bool changed = *newVals.first != *vals.first ||
274 *newVals.second != *vals.second;
275 *vals.first = *newVals.first;
276 *vals.second = *newVals.second;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000277#if 0
278 printVector(block, *vals.first, 1);
279 printVector(block, *vals.second, 2);
280#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000281 return changed;
282}
283
284void CFGBlockValues::resetScratch() {
285 scratch.reset();
286}
287
Ted Kremenek136f8f22011-03-15 04:57:27 +0000288ValueVector::reference CFGBlockValues::operator[](const VarDecl *vd) {
289 const llvm::Optional<unsigned> &idx = DeclToIndex.getValueIndex(vd);
Ted Kremenek610068c2011-01-15 02:58:47 +0000290 assert(idx.hasValue());
291 return scratch[idx.getValue()];
292}
293
294//------------------------------------------------------------------------====//
295// Worklist: worklist for dataflow analysis.
296//====------------------------------------------------------------------------//
297
298namespace {
299class DataflowWorklist {
300 llvm::SmallVector<const CFGBlock *, 20> worklist;
Ted Kremenek496398d2011-03-15 04:57:32 +0000301 llvm::BitVector enqueuedBlocks;
Ted Kremenek610068c2011-01-15 02:58:47 +0000302public:
303 DataflowWorklist(const CFG &cfg) : enqueuedBlocks(cfg.getNumBlockIDs()) {}
304
305 void enqueue(const CFGBlock *block);
306 void enqueueSuccessors(const CFGBlock *block);
307 const CFGBlock *dequeue();
308
309};
310}
311
312void DataflowWorklist::enqueue(const CFGBlock *block) {
Ted Kremenekc104e532011-01-18 04:53:25 +0000313 if (!block)
314 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000315 unsigned idx = block->getBlockID();
316 if (enqueuedBlocks[idx])
317 return;
318 worklist.push_back(block);
319 enqueuedBlocks[idx] = true;
320}
321
322void DataflowWorklist::enqueueSuccessors(const clang::CFGBlock *block) {
323 for (CFGBlock::const_succ_iterator I = block->succ_begin(),
324 E = block->succ_end(); I != E; ++I) {
325 enqueue(*I);
326 }
327}
328
329const CFGBlock *DataflowWorklist::dequeue() {
330 if (worklist.empty())
331 return 0;
332 const CFGBlock *b = worklist.back();
333 worklist.pop_back();
334 enqueuedBlocks[b->getBlockID()] = false;
335 return b;
336}
337
338//------------------------------------------------------------------------====//
339// Transfer function for uninitialized values analysis.
340//====------------------------------------------------------------------------//
341
Ted Kremenek610068c2011-01-15 02:58:47 +0000342namespace {
343class FindVarResult {
344 const VarDecl *vd;
345 const DeclRefExpr *dr;
346public:
347 FindVarResult(VarDecl *vd, DeclRefExpr *dr) : vd(vd), dr(dr) {}
348
349 const DeclRefExpr *getDeclRefExpr() const { return dr; }
350 const VarDecl *getDecl() const { return vd; }
351};
352
353class TransferFunctions : public CFGRecStmtVisitor<TransferFunctions> {
354 CFGBlockValues &vals;
355 const CFG &cfg;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000356 AnalysisContext &ac;
Ted Kremenek610068c2011-01-15 02:58:47 +0000357 UninitVariablesHandler *handler;
Ted Kremenekc21fed32011-01-18 21:18:58 +0000358 const DeclRefExpr *currentDR;
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000359 const Expr *currentVoidCast;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000360 const bool flagBlockUses;
Ted Kremenek610068c2011-01-15 02:58:47 +0000361public:
362 TransferFunctions(CFGBlockValues &vals, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000363 AnalysisContext &ac,
364 UninitVariablesHandler *handler,
365 bool flagBlockUses)
366 : vals(vals), cfg(cfg), ac(ac), handler(handler), currentDR(0),
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000367 currentVoidCast(0), flagBlockUses(flagBlockUses) {}
Ted Kremenek610068c2011-01-15 02:58:47 +0000368
369 const CFG &getCFG() { return cfg; }
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000370 void reportUninit(const DeclRefExpr *ex, const VarDecl *vd,
371 bool isAlwaysUninit);
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000372
373 void VisitBlockExpr(BlockExpr *be);
Ted Kremenek610068c2011-01-15 02:58:47 +0000374 void VisitDeclStmt(DeclStmt *ds);
Ted Kremenekc21fed32011-01-18 21:18:58 +0000375 void VisitDeclRefExpr(DeclRefExpr *dr);
Ted Kremenek610068c2011-01-15 02:58:47 +0000376 void VisitUnaryOperator(UnaryOperator *uo);
377 void VisitBinaryOperator(BinaryOperator *bo);
378 void VisitCastExpr(CastExpr *ce);
Peter Collingbournef4e3cfb2011-03-11 19:24:49 +0000379 void VisitUnaryExprOrTypeTraitExpr(UnaryExprOrTypeTraitExpr *se);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000380 void BlockStmt_VisitObjCForCollectionStmt(ObjCForCollectionStmt *fs);
Ted Kremenek40900ee2011-01-27 02:29:34 +0000381
382 bool isTrackedVar(const VarDecl *vd) {
383 return ::isTrackedVar(vd, cast<DeclContext>(ac.getDecl()));
384 }
385
386 FindVarResult findBlockVarDecl(Expr *ex);
Ted Kremenek610068c2011-01-15 02:58:47 +0000387};
388}
389
390void TransferFunctions::reportUninit(const DeclRefExpr *ex,
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000391 const VarDecl *vd, bool isAlwaysUnit) {
392 if (handler) handler->handleUseOfUninitVariable(ex, vd, isAlwaysUnit);
Ted Kremenek610068c2011-01-15 02:58:47 +0000393}
394
Ted Kremenek40900ee2011-01-27 02:29:34 +0000395FindVarResult TransferFunctions::findBlockVarDecl(Expr* ex) {
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000396 if (DeclRefExpr* dr = dyn_cast<DeclRefExpr>(ex->IgnoreParenCasts()))
397 if (VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
398 if (isTrackedVar(vd))
Ted Kremenek40900ee2011-01-27 02:29:34 +0000399 return FindVarResult(vd, dr);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000400 return FindVarResult(0, 0);
401}
402
403void TransferFunctions::BlockStmt_VisitObjCForCollectionStmt(
404 ObjCForCollectionStmt *fs) {
405
406 Visit(fs->getCollection());
407
408 // This represents an initialization of the 'element' value.
409 Stmt *element = fs->getElement();
410 const VarDecl* vd = 0;
411
412 if (DeclStmt* ds = dyn_cast<DeclStmt>(element)) {
413 vd = cast<VarDecl>(ds->getSingleDecl());
414 if (!isTrackedVar(vd))
415 vd = 0;
416 }
417 else {
418 // Initialize the value of the reference variable.
419 const FindVarResult &res = findBlockVarDecl(cast<Expr>(element));
420 vd = res.getDecl();
421 if (!vd) {
422 Visit(element);
423 return;
424 }
425 }
426
427 if (vd)
428 vals[vd] = Initialized;
429}
430
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000431void TransferFunctions::VisitBlockExpr(BlockExpr *be) {
432 if (!flagBlockUses || !handler)
433 return;
434 AnalysisContext::referenced_decls_iterator i, e;
435 llvm::tie(i, e) = ac.getReferencedBlockVars(be->getBlockDecl());
436 for ( ; i != e; ++i) {
437 const VarDecl *vd = *i;
Ted Kremenek40900ee2011-01-27 02:29:34 +0000438 if (vd->getAttr<BlocksAttr>() || !vd->hasLocalStorage() ||
439 !isTrackedVar(vd))
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000440 continue;
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000441 Value v = vals[vd];
442 if (isUninitialized(v))
443 handler->handleUseOfUninitVariable(be, vd, isAlwaysUninit(v));
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000444 }
445}
446
Ted Kremenek610068c2011-01-15 02:58:47 +0000447void TransferFunctions::VisitDeclStmt(DeclStmt *ds) {
448 for (DeclStmt::decl_iterator DI = ds->decl_begin(), DE = ds->decl_end();
449 DI != DE; ++DI) {
450 if (VarDecl *vd = dyn_cast<VarDecl>(*DI)) {
Ted Kremenek4dccb902011-01-18 05:00:42 +0000451 if (isTrackedVar(vd)) {
452 vals[vd] = Uninitialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000453 if (Stmt *init = vd->getInit()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000454 Visit(init);
Ted Kremenekc104e532011-01-18 04:53:25 +0000455 vals[vd] = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000456 }
Ted Kremenek4dccb902011-01-18 05:00:42 +0000457 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000458 else if (Stmt *init = vd->getInit()) {
459 Visit(init);
460 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000461 }
462 }
463}
464
Ted Kremenekc21fed32011-01-18 21:18:58 +0000465void TransferFunctions::VisitDeclRefExpr(DeclRefExpr *dr) {
466 // We assume that DeclRefExprs wrapped in an lvalue-to-rvalue cast
467 // cannot be block-level expressions. Therefore, we determine if
468 // a DeclRefExpr is involved in a "load" by comparing it to the current
469 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
470 // If a DeclRefExpr is not involved in a load, we are essentially computing
471 // its address, either for assignment to a reference or via the '&' operator.
472 // In such cases, treat the variable as being initialized, since this
473 // analysis isn't powerful enough to do alias tracking.
474 if (dr != currentDR)
475 if (const VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
476 if (isTrackedVar(vd))
477 vals[vd] = Initialized;
478}
479
Ted Kremenek610068c2011-01-15 02:58:47 +0000480void TransferFunctions::VisitBinaryOperator(clang::BinaryOperator *bo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000481 if (bo->isAssignmentOp()) {
482 const FindVarResult &res = findBlockVarDecl(bo->getLHS());
483 if (const VarDecl* vd = res.getDecl()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000484 // We assume that DeclRefExprs wrapped in a BinaryOperator "assignment"
485 // cannot be block-level expressions. Therefore, we determine if
486 // a DeclRefExpr is involved in a "load" by comparing it to the current
487 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
488 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
489 res.getDeclRefExpr());
490 Visit(bo->getRHS());
491 Visit(bo->getLHS());
492
Ted Kremenek496398d2011-03-15 04:57:32 +0000493 ValueVector::reference val = vals[vd];
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000494 if (isUninitialized(val)) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000495 if (bo->getOpcode() != BO_Assign)
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000496 reportUninit(res.getDeclRefExpr(), vd, isAlwaysUninit(val));
Ted Kremenek496398d2011-03-15 04:57:32 +0000497 val = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000498 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000499 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000500 }
501 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000502 Visit(bo->getRHS());
503 Visit(bo->getLHS());
Ted Kremenek610068c2011-01-15 02:58:47 +0000504}
505
506void TransferFunctions::VisitUnaryOperator(clang::UnaryOperator *uo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000507 switch (uo->getOpcode()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000508 case clang::UO_PostDec:
509 case clang::UO_PostInc:
510 case clang::UO_PreDec:
511 case clang::UO_PreInc: {
512 const FindVarResult &res = findBlockVarDecl(uo->getSubExpr());
513 if (const VarDecl *vd = res.getDecl()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000514 // We assume that DeclRefExprs wrapped in a unary operator ++/--
515 // cannot be block-level expressions. Therefore, we determine if
516 // a DeclRefExpr is involved in a "load" by comparing it to the current
517 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
518 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
519 res.getDeclRefExpr());
520 Visit(uo->getSubExpr());
521
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000522 ValueVector::reference val = vals[vd];
523 if (isUninitialized(val)) {
524 reportUninit(res.getDeclRefExpr(), vd, isAlwaysUninit(val));
525 // Don't cascade warnings.
526 val = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000527 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000528 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000529 }
530 break;
531 }
532 default:
533 break;
534 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000535 Visit(uo->getSubExpr());
Ted Kremenek610068c2011-01-15 02:58:47 +0000536}
537
538void TransferFunctions::VisitCastExpr(clang::CastExpr *ce) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000539 if (ce->getCastKind() == CK_LValueToRValue) {
540 const FindVarResult &res = findBlockVarDecl(ce->getSubExpr());
Ted Kremenekc21fed32011-01-18 21:18:58 +0000541 if (const VarDecl *vd = res.getDecl()) {
542 // We assume that DeclRefExprs wrapped in an lvalue-to-rvalue cast
543 // cannot be block-level expressions. Therefore, we determine if
544 // a DeclRefExpr is involved in a "load" by comparing it to the current
545 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
546 // Here we update 'currentDR' to be the one associated with this
547 // lvalue-to-rvalue cast. Then, when we analyze the DeclRefExpr, we
548 // will know that we are not computing its lvalue for other purposes
549 // than to perform a load.
550 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
551 res.getDeclRefExpr());
552 Visit(ce->getSubExpr());
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000553 if (currentVoidCast != ce) {
554 Value val = vals[vd];
555 if (isUninitialized(val)) {
556 reportUninit(res.getDeclRefExpr(), vd, isAlwaysUninit(val));
557 // Don't cascade warnings.
558 vals[vd] = Initialized;
559 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000560 }
561 return;
562 }
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000563 }
564 else if (CStyleCastExpr *cse = dyn_cast<CStyleCastExpr>(ce)) {
565 if (cse->getType()->isVoidType()) {
566 // e.g. (void) x;
567 SaveAndRestore<const Expr *>
568 lastVoidCast(currentVoidCast, cse->getSubExpr()->IgnoreParens());
569 Visit(cse->getSubExpr());
570 return;
571 }
572 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000573 Visit(ce->getSubExpr());
Ted Kremenek610068c2011-01-15 02:58:47 +0000574}
575
Peter Collingbournef4e3cfb2011-03-11 19:24:49 +0000576void TransferFunctions::VisitUnaryExprOrTypeTraitExpr(
577 UnaryExprOrTypeTraitExpr *se) {
578 if (se->getKind() == UETT_SizeOf) {
Ted Kremenek96608032011-01-23 17:53:04 +0000579 if (se->getType()->isConstantSizeType())
580 return;
581 // Handle VLAs.
582 Visit(se->getArgumentExpr());
583 }
584}
585
Ted Kremenek610068c2011-01-15 02:58:47 +0000586//------------------------------------------------------------------------====//
587// High-level "driver" logic for uninitialized values analysis.
588//====------------------------------------------------------------------------//
589
Ted Kremenek13bd4232011-01-20 17:37:17 +0000590static bool runOnBlock(const CFGBlock *block, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000591 AnalysisContext &ac, CFGBlockValues &vals,
592 UninitVariablesHandler *handler = 0,
593 bool flagBlockUses = false) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000594
595 if (const BinaryOperator *b = getLogicalOperatorInChain(block)) {
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000596 CFGBlock::const_pred_iterator itr = block->pred_begin();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000597 BVPair vA = vals.getValueVectors(*itr, false);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000598 ++itr;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000599 BVPair vB = vals.getValueVectors(*itr, false);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000600
601 BVPair valsAB;
602
603 if (b->getOpcode() == BO_LAnd) {
604 // Merge the 'F' bits from the first and second.
605 vals.mergeIntoScratch(*(vA.second ? vA.second : vA.first), true);
606 vals.mergeIntoScratch(*(vB.second ? vB.second : vB.first), false);
607 valsAB.first = vA.first;
Ted Kremenek2d4bed12011-01-20 21:25:31 +0000608 valsAB.second = &vals.getScratch();
Ted Kremenek13bd4232011-01-20 17:37:17 +0000609 }
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000610 else {
611 // Merge the 'T' bits from the first and second.
612 assert(b->getOpcode() == BO_LOr);
613 vals.mergeIntoScratch(*vA.first, true);
614 vals.mergeIntoScratch(*vB.first, false);
615 valsAB.first = &vals.getScratch();
616 valsAB.second = vA.second ? vA.second : vA.first;
617 }
Ted Kremenek136f8f22011-03-15 04:57:27 +0000618 return vals.updateValueVectors(block, valsAB);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000619 }
620
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000621 // Default behavior: merge in values of predecessor blocks.
Ted Kremenek610068c2011-01-15 02:58:47 +0000622 vals.resetScratch();
623 bool isFirst = true;
624 for (CFGBlock::const_pred_iterator I = block->pred_begin(),
625 E = block->pred_end(); I != E; ++I) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000626 vals.mergeIntoScratch(vals.getValueVector(*I, block), isFirst);
Ted Kremenek610068c2011-01-15 02:58:47 +0000627 isFirst = false;
628 }
629 // Apply the transfer function.
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000630 TransferFunctions tf(vals, cfg, ac, handler, flagBlockUses);
Ted Kremenek610068c2011-01-15 02:58:47 +0000631 for (CFGBlock::const_iterator I = block->begin(), E = block->end();
632 I != E; ++I) {
633 if (const CFGStmt *cs = dyn_cast<CFGStmt>(&*I)) {
634 tf.BlockStmt_Visit(cs->getStmt());
635 }
636 }
Ted Kremenek136f8f22011-03-15 04:57:27 +0000637 return vals.updateValueVectorWithScratch(block);
Ted Kremenek610068c2011-01-15 02:58:47 +0000638}
639
640void clang::runUninitializedVariablesAnalysis(const DeclContext &dc,
641 const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000642 AnalysisContext &ac,
Ted Kremenek610068c2011-01-15 02:58:47 +0000643 UninitVariablesHandler &handler) {
644 CFGBlockValues vals(cfg);
645 vals.computeSetOfDeclarations(dc);
646 if (vals.hasNoDeclarations())
647 return;
648 DataflowWorklist worklist(cfg);
Ted Kremenek496398d2011-03-15 04:57:32 +0000649 llvm::BitVector previouslyVisited(cfg.getNumBlockIDs());
Ted Kremenek610068c2011-01-15 02:58:47 +0000650
651 worklist.enqueueSuccessors(&cfg.getEntry());
652
653 while (const CFGBlock *block = worklist.dequeue()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000654 // Did the block change?
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000655 bool changed = runOnBlock(block, cfg, ac, vals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000656 if (changed || !previouslyVisited[block->getBlockID()])
657 worklist.enqueueSuccessors(block);
658 previouslyVisited[block->getBlockID()] = true;
659 }
660
661 // Run through the blocks one more time, and report uninitialized variabes.
662 for (CFG::const_iterator BI = cfg.begin(), BE = cfg.end(); BI != BE; ++BI) {
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000663 runOnBlock(*BI, cfg, ac, vals, &handler, /* flagBlockUses */ true);
Ted Kremenek610068c2011-01-15 02:58:47 +0000664 }
665}
666
667UninitVariablesHandler::~UninitVariablesHandler() {}
668