blob: 75eccbf7a3c80da33f8f4c911e931d46daa8f688 [file] [log] [blame]
Ted Kremenek610068c2011-01-15 02:58:47 +00001//==- UninitializedValuesV2.cpp - Find Uninitialized Values -----*- C++ --*-==//
2//
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"
23#include "clang/Analysis/Analyses/UninitializedValuesV2.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//------------------------------------------------------------------------====//
35// DeclToBit: a mapping from Decls we track to bitvector indices.
36//====------------------------------------------------------------------------//
37
38namespace {
39class DeclToBit {
40 llvm::DenseMap<const VarDecl *, unsigned> map;
41public:
42 DeclToBit() {}
43
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.
51 llvm::Optional<unsigned> getBitVectorIndex(const VarDecl *d);
52};
53}
54
55void DeclToBit::computeMap(const DeclContext &dc) {
56 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
66llvm::Optional<unsigned> DeclToBit::getBitVectorIndex(const VarDecl *d) {
67 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 Kremenek13bd4232011-01-20 17:37:17 +000077typedef std::pair<llvm::BitVector *, llvm::BitVector *> BVPair;
78
Ted Kremenek610068c2011-01-15 02:58:47 +000079namespace {
80class CFGBlockValues {
81 const CFG &cfg;
Ted Kremenek13bd4232011-01-20 17:37:17 +000082 BVPair *vals;
Ted Kremenek610068c2011-01-15 02:58:47 +000083 llvm::BitVector scratch;
84 DeclToBit declToBit;
Ted Kremenek13bd4232011-01-20 17:37:17 +000085
86 llvm::BitVector &lazyCreate(llvm::BitVector *&bv);
Ted Kremenek610068c2011-01-15 02:58:47 +000087public:
88 CFGBlockValues(const CFG &cfg);
89 ~CFGBlockValues();
90
91 void computeSetOfDeclarations(const DeclContext &dc);
Ted Kremenek13bd4232011-01-20 17:37:17 +000092 llvm::BitVector &getBitVector(const CFGBlock *block,
93 const CFGBlock *dstBlock);
94
Ted Kremenek9fcbcee2011-02-01 17:43:18 +000095 BVPair &getBitVectors(const CFGBlock *block, bool shouldLazyCreate);
Ted Kremenek13bd4232011-01-20 17:37:17 +000096
Ted Kremenek610068c2011-01-15 02:58:47 +000097 void mergeIntoScratch(llvm::BitVector const &source, bool isFirst);
98 bool updateBitVectorWithScratch(const CFGBlock *block);
Ted Kremenek13bd4232011-01-20 17:37:17 +000099 bool updateBitVectors(const CFGBlock *block, const BVPair &newVals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000100
101 bool hasNoDeclarations() const {
102 return declToBit.size() == 0;
103 }
104
105 void resetScratch();
Ted Kremenek13bd4232011-01-20 17:37:17 +0000106 llvm::BitVector &getScratch() { return scratch; }
107
Ted Kremenek610068c2011-01-15 02:58:47 +0000108 llvm::BitVector::reference operator[](const VarDecl *vd);
109};
110}
111
112CFGBlockValues::CFGBlockValues(const CFG &c) : cfg(c), vals(0) {
113 unsigned n = cfg.getNumBlockIDs();
114 if (!n)
115 return;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000116 vals = new std::pair<llvm::BitVector*, llvm::BitVector*>[n];
Francois Pichet2d78c372011-01-15 13:27:47 +0000117 memset(vals, 0, sizeof(*vals) * n);
Ted Kremenek610068c2011-01-15 02:58:47 +0000118}
119
120CFGBlockValues::~CFGBlockValues() {
121 unsigned n = cfg.getNumBlockIDs();
122 if (n == 0)
123 return;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000124 for (unsigned i = 0; i < n; ++i) {
125 delete vals[i].first;
126 delete vals[i].second;
127 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000128 delete [] vals;
129}
130
131void CFGBlockValues::computeSetOfDeclarations(const DeclContext &dc) {
132 declToBit.computeMap(dc);
133 scratch.resize(declToBit.size());
134}
135
Ted Kremenek13bd4232011-01-20 17:37:17 +0000136llvm::BitVector &CFGBlockValues::lazyCreate(llvm::BitVector *&bv) {
137 if (!bv)
Ted Kremenek610068c2011-01-15 02:58:47 +0000138 bv = new llvm::BitVector(declToBit.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000139 return *bv;
140}
141
Ted Kremenek13bd4232011-01-20 17:37:17 +0000142/// This function pattern matches for a '&&' or '||' that appears at
143/// the beginning of a CFGBlock that also (1) has a terminator and
144/// (2) has no other elements. If such an expression is found, it is returned.
145static BinaryOperator *getLogicalOperatorInChain(const CFGBlock *block) {
146 if (block->empty())
147 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000148
Ted Kremenek13bd4232011-01-20 17:37:17 +0000149 CFGStmt cstmt = block->front().getAs<CFGStmt>();
150 BinaryOperator *b = llvm::dyn_cast_or_null<BinaryOperator>(cstmt.getStmt());
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000151
152 if (!b || !b->isLogicalOp())
Ted Kremenek13bd4232011-01-20 17:37:17 +0000153 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000154
155 if (block->pred_size() == 2 &&
156 ((block->succ_size() == 2 && block->getTerminatorCondition() == b) ||
157 block->size() == 1))
158 return b;
159
160 return 0;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000161}
162
163llvm::BitVector &CFGBlockValues::getBitVector(const CFGBlock *block,
164 const CFGBlock *dstBlock) {
165 unsigned idx = block->getBlockID();
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000166 if (dstBlock && getLogicalOperatorInChain(block)) {
167 if (*block->succ_begin() == dstBlock)
168 return lazyCreate(vals[idx].first);
169 assert(*(block->succ_begin()+1) == dstBlock);
170 return lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000171 }
172
173 assert(vals[idx].second == 0);
174 return lazyCreate(vals[idx].first);
175}
176
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000177BVPair &CFGBlockValues::getBitVectors(const clang::CFGBlock *block,
178 bool shouldLazyCreate) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000179 unsigned idx = block->getBlockID();
180 lazyCreate(vals[idx].first);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000181 if (shouldLazyCreate)
182 lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000183 return vals[idx];
184}
185
Ted Kremenek610068c2011-01-15 02:58:47 +0000186void CFGBlockValues::mergeIntoScratch(llvm::BitVector const &source,
187 bool isFirst) {
188 if (isFirst)
189 scratch = source;
190 else
Ted Kremenekc104e532011-01-18 04:53:25 +0000191 scratch |= source;
Ted Kremenek610068c2011-01-15 02:58:47 +0000192}
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000193#if 0
194static void printVector(const CFGBlock *block, llvm::BitVector &bv,
195 unsigned num) {
196
197 llvm::errs() << block->getBlockID() << " :";
198 for (unsigned i = 0; i < bv.size(); ++i) {
199 llvm::errs() << ' ' << bv[i];
200 }
201 llvm::errs() << " : " << num << '\n';
202}
203#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000204
205bool CFGBlockValues::updateBitVectorWithScratch(const CFGBlock *block) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000206 llvm::BitVector &dst = getBitVector(block, 0);
Ted Kremenek610068c2011-01-15 02:58:47 +0000207 bool changed = (dst != scratch);
208 if (changed)
209 dst = scratch;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000210#if 0
211 printVector(block, scratch, 0);
212#endif
Ted Kremenek13bd4232011-01-20 17:37:17 +0000213 return changed;
214}
215
216bool CFGBlockValues::updateBitVectors(const CFGBlock *block,
217 const BVPair &newVals) {
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000218 BVPair &vals = getBitVectors(block, true);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000219 bool changed = *newVals.first != *vals.first ||
220 *newVals.second != *vals.second;
221 *vals.first = *newVals.first;
222 *vals.second = *newVals.second;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000223#if 0
224 printVector(block, *vals.first, 1);
225 printVector(block, *vals.second, 2);
226#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000227 return changed;
228}
229
230void CFGBlockValues::resetScratch() {
231 scratch.reset();
232}
233
234llvm::BitVector::reference CFGBlockValues::operator[](const VarDecl *vd) {
235 const llvm::Optional<unsigned> &idx = declToBit.getBitVectorIndex(vd);
236 assert(idx.hasValue());
237 return scratch[idx.getValue()];
238}
239
240//------------------------------------------------------------------------====//
241// Worklist: worklist for dataflow analysis.
242//====------------------------------------------------------------------------//
243
244namespace {
245class DataflowWorklist {
246 llvm::SmallVector<const CFGBlock *, 20> worklist;
247 llvm::BitVector enqueuedBlocks;
248public:
249 DataflowWorklist(const CFG &cfg) : enqueuedBlocks(cfg.getNumBlockIDs()) {}
250
251 void enqueue(const CFGBlock *block);
252 void enqueueSuccessors(const CFGBlock *block);
253 const CFGBlock *dequeue();
254
255};
256}
257
258void DataflowWorklist::enqueue(const CFGBlock *block) {
Ted Kremenekc104e532011-01-18 04:53:25 +0000259 if (!block)
260 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000261 unsigned idx = block->getBlockID();
262 if (enqueuedBlocks[idx])
263 return;
264 worklist.push_back(block);
265 enqueuedBlocks[idx] = true;
266}
267
268void DataflowWorklist::enqueueSuccessors(const clang::CFGBlock *block) {
269 for (CFGBlock::const_succ_iterator I = block->succ_begin(),
270 E = block->succ_end(); I != E; ++I) {
271 enqueue(*I);
272 }
273}
274
275const CFGBlock *DataflowWorklist::dequeue() {
276 if (worklist.empty())
277 return 0;
278 const CFGBlock *b = worklist.back();
279 worklist.pop_back();
280 enqueuedBlocks[b->getBlockID()] = false;
281 return b;
282}
283
284//------------------------------------------------------------------------====//
285// Transfer function for uninitialized values analysis.
286//====------------------------------------------------------------------------//
287
Ted Kremenekc104e532011-01-18 04:53:25 +0000288static const bool Initialized = false;
289static const bool Uninitialized = true;
Ted Kremenek610068c2011-01-15 02:58:47 +0000290
291namespace {
292class FindVarResult {
293 const VarDecl *vd;
294 const DeclRefExpr *dr;
295public:
296 FindVarResult(VarDecl *vd, DeclRefExpr *dr) : vd(vd), dr(dr) {}
297
298 const DeclRefExpr *getDeclRefExpr() const { return dr; }
299 const VarDecl *getDecl() const { return vd; }
300};
301
302class TransferFunctions : public CFGRecStmtVisitor<TransferFunctions> {
303 CFGBlockValues &vals;
304 const CFG &cfg;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000305 AnalysisContext &ac;
Ted Kremenek610068c2011-01-15 02:58:47 +0000306 UninitVariablesHandler *handler;
Ted Kremenekc21fed32011-01-18 21:18:58 +0000307 const DeclRefExpr *currentDR;
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000308 const Expr *currentVoidCast;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000309 const bool flagBlockUses;
Ted Kremenek610068c2011-01-15 02:58:47 +0000310public:
311 TransferFunctions(CFGBlockValues &vals, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000312 AnalysisContext &ac,
313 UninitVariablesHandler *handler,
314 bool flagBlockUses)
315 : vals(vals), cfg(cfg), ac(ac), handler(handler), currentDR(0),
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000316 currentVoidCast(0), flagBlockUses(flagBlockUses) {}
Ted Kremenek610068c2011-01-15 02:58:47 +0000317
318 const CFG &getCFG() { return cfg; }
319 void reportUninit(const DeclRefExpr *ex, const VarDecl *vd);
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000320
321 void VisitBlockExpr(BlockExpr *be);
Ted Kremenek610068c2011-01-15 02:58:47 +0000322 void VisitDeclStmt(DeclStmt *ds);
Ted Kremenekc21fed32011-01-18 21:18:58 +0000323 void VisitDeclRefExpr(DeclRefExpr *dr);
Ted Kremenek610068c2011-01-15 02:58:47 +0000324 void VisitUnaryOperator(UnaryOperator *uo);
325 void VisitBinaryOperator(BinaryOperator *bo);
326 void VisitCastExpr(CastExpr *ce);
Ted Kremenek96608032011-01-23 17:53:04 +0000327 void VisitSizeOfAlignOfExpr(SizeOfAlignOfExpr *se);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000328 void BlockStmt_VisitObjCForCollectionStmt(ObjCForCollectionStmt *fs);
Ted Kremenek40900ee2011-01-27 02:29:34 +0000329
330 bool isTrackedVar(const VarDecl *vd) {
331 return ::isTrackedVar(vd, cast<DeclContext>(ac.getDecl()));
332 }
333
334 FindVarResult findBlockVarDecl(Expr *ex);
Ted Kremenek610068c2011-01-15 02:58:47 +0000335};
336}
337
338void TransferFunctions::reportUninit(const DeclRefExpr *ex,
339 const VarDecl *vd) {
340 if (handler) handler->handleUseOfUninitVariable(ex, vd);
341}
342
Ted Kremenek40900ee2011-01-27 02:29:34 +0000343FindVarResult TransferFunctions::findBlockVarDecl(Expr* ex) {
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000344 if (DeclRefExpr* dr = dyn_cast<DeclRefExpr>(ex->IgnoreParenCasts()))
345 if (VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
346 if (isTrackedVar(vd))
Ted Kremenek40900ee2011-01-27 02:29:34 +0000347 return FindVarResult(vd, dr);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000348 return FindVarResult(0, 0);
349}
350
351void TransferFunctions::BlockStmt_VisitObjCForCollectionStmt(
352 ObjCForCollectionStmt *fs) {
353
354 Visit(fs->getCollection());
355
356 // This represents an initialization of the 'element' value.
357 Stmt *element = fs->getElement();
358 const VarDecl* vd = 0;
359
360 if (DeclStmt* ds = dyn_cast<DeclStmt>(element)) {
361 vd = cast<VarDecl>(ds->getSingleDecl());
362 if (!isTrackedVar(vd))
363 vd = 0;
364 }
365 else {
366 // Initialize the value of the reference variable.
367 const FindVarResult &res = findBlockVarDecl(cast<Expr>(element));
368 vd = res.getDecl();
369 if (!vd) {
370 Visit(element);
371 return;
372 }
373 }
374
375 if (vd)
376 vals[vd] = Initialized;
377}
378
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000379void TransferFunctions::VisitBlockExpr(BlockExpr *be) {
380 if (!flagBlockUses || !handler)
381 return;
382 AnalysisContext::referenced_decls_iterator i, e;
383 llvm::tie(i, e) = ac.getReferencedBlockVars(be->getBlockDecl());
384 for ( ; i != e; ++i) {
385 const VarDecl *vd = *i;
Ted Kremenek40900ee2011-01-27 02:29:34 +0000386 if (vd->getAttr<BlocksAttr>() || !vd->hasLocalStorage() ||
387 !isTrackedVar(vd))
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000388 continue;
389 if (vals[vd] == Uninitialized)
Ted Kremenek40900ee2011-01-27 02:29:34 +0000390 handler->handleUseOfUninitVariable(be, vd);
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000391 }
392}
393
Ted Kremenek610068c2011-01-15 02:58:47 +0000394void TransferFunctions::VisitDeclStmt(DeclStmt *ds) {
395 for (DeclStmt::decl_iterator DI = ds->decl_begin(), DE = ds->decl_end();
396 DI != DE; ++DI) {
397 if (VarDecl *vd = dyn_cast<VarDecl>(*DI)) {
Ted Kremenek4dccb902011-01-18 05:00:42 +0000398 if (isTrackedVar(vd)) {
399 vals[vd] = Uninitialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000400 if (Stmt *init = vd->getInit()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000401 Visit(init);
Ted Kremenekc104e532011-01-18 04:53:25 +0000402 vals[vd] = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000403 }
Ted Kremenek4dccb902011-01-18 05:00:42 +0000404 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000405 else if (Stmt *init = vd->getInit()) {
406 Visit(init);
407 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000408 }
409 }
410}
411
Ted Kremenekc21fed32011-01-18 21:18:58 +0000412void TransferFunctions::VisitDeclRefExpr(DeclRefExpr *dr) {
413 // We assume that DeclRefExprs wrapped in an lvalue-to-rvalue cast
414 // cannot be block-level expressions. Therefore, we determine if
415 // a DeclRefExpr is involved in a "load" by comparing it to the current
416 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
417 // If a DeclRefExpr is not involved in a load, we are essentially computing
418 // its address, either for assignment to a reference or via the '&' operator.
419 // In such cases, treat the variable as being initialized, since this
420 // analysis isn't powerful enough to do alias tracking.
421 if (dr != currentDR)
422 if (const VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
423 if (isTrackedVar(vd))
424 vals[vd] = Initialized;
425}
426
Ted Kremenek610068c2011-01-15 02:58:47 +0000427void TransferFunctions::VisitBinaryOperator(clang::BinaryOperator *bo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000428 if (bo->isAssignmentOp()) {
429 const FindVarResult &res = findBlockVarDecl(bo->getLHS());
430 if (const VarDecl* vd = res.getDecl()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000431 // We assume that DeclRefExprs wrapped in a BinaryOperator "assignment"
432 // cannot be block-level expressions. Therefore, we determine if
433 // a DeclRefExpr is involved in a "load" by comparing it to the current
434 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
435 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
436 res.getDeclRefExpr());
437 Visit(bo->getRHS());
438 Visit(bo->getLHS());
439
Ted Kremenek610068c2011-01-15 02:58:47 +0000440 llvm::BitVector::reference bit = vals[vd];
441 if (bit == Uninitialized) {
442 if (bo->getOpcode() != BO_Assign)
443 reportUninit(res.getDeclRefExpr(), vd);
444 bit = Initialized;
445 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000446 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000447 }
448 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000449 Visit(bo->getRHS());
450 Visit(bo->getLHS());
Ted Kremenek610068c2011-01-15 02:58:47 +0000451}
452
453void TransferFunctions::VisitUnaryOperator(clang::UnaryOperator *uo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000454 switch (uo->getOpcode()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000455 case clang::UO_PostDec:
456 case clang::UO_PostInc:
457 case clang::UO_PreDec:
458 case clang::UO_PreInc: {
459 const FindVarResult &res = findBlockVarDecl(uo->getSubExpr());
460 if (const VarDecl *vd = res.getDecl()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000461 // We assume that DeclRefExprs wrapped in a unary operator ++/--
462 // cannot be block-level expressions. Therefore, we determine if
463 // a DeclRefExpr is involved in a "load" by comparing it to the current
464 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
465 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
466 res.getDeclRefExpr());
467 Visit(uo->getSubExpr());
468
Ted Kremenek610068c2011-01-15 02:58:47 +0000469 llvm::BitVector::reference bit = vals[vd];
470 if (bit == Uninitialized) {
471 reportUninit(res.getDeclRefExpr(), vd);
472 bit = Initialized;
473 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000474 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000475 }
476 break;
477 }
478 default:
479 break;
480 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000481 Visit(uo->getSubExpr());
Ted Kremenek610068c2011-01-15 02:58:47 +0000482}
483
484void TransferFunctions::VisitCastExpr(clang::CastExpr *ce) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000485 if (ce->getCastKind() == CK_LValueToRValue) {
486 const FindVarResult &res = findBlockVarDecl(ce->getSubExpr());
Ted Kremenekc21fed32011-01-18 21:18:58 +0000487 if (const VarDecl *vd = res.getDecl()) {
488 // We assume that DeclRefExprs wrapped in an lvalue-to-rvalue cast
489 // cannot be block-level expressions. Therefore, we determine if
490 // a DeclRefExpr is involved in a "load" by comparing it to the current
491 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
492 // Here we update 'currentDR' to be the one associated with this
493 // lvalue-to-rvalue cast. Then, when we analyze the DeclRefExpr, we
494 // will know that we are not computing its lvalue for other purposes
495 // than to perform a load.
496 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
497 res.getDeclRefExpr());
498 Visit(ce->getSubExpr());
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000499 if (currentVoidCast != ce && vals[vd] == Uninitialized) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000500 reportUninit(res.getDeclRefExpr(), vd);
Ted Kremenekc21fed32011-01-18 21:18:58 +0000501 // Don't cascade warnings.
502 vals[vd] = Initialized;
503 }
504 return;
505 }
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000506 }
507 else if (CStyleCastExpr *cse = dyn_cast<CStyleCastExpr>(ce)) {
508 if (cse->getType()->isVoidType()) {
509 // e.g. (void) x;
510 SaveAndRestore<const Expr *>
511 lastVoidCast(currentVoidCast, cse->getSubExpr()->IgnoreParens());
512 Visit(cse->getSubExpr());
513 return;
514 }
515 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000516 Visit(ce->getSubExpr());
Ted Kremenek610068c2011-01-15 02:58:47 +0000517}
518
Ted Kremenek96608032011-01-23 17:53:04 +0000519void TransferFunctions::VisitSizeOfAlignOfExpr(SizeOfAlignOfExpr *se) {
520 if (se->isSizeOf()) {
521 if (se->getType()->isConstantSizeType())
522 return;
523 // Handle VLAs.
524 Visit(se->getArgumentExpr());
525 }
526}
527
Ted Kremenek610068c2011-01-15 02:58:47 +0000528//------------------------------------------------------------------------====//
529// High-level "driver" logic for uninitialized values analysis.
530//====------------------------------------------------------------------------//
531
Ted Kremenek13bd4232011-01-20 17:37:17 +0000532static bool runOnBlock(const CFGBlock *block, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000533 AnalysisContext &ac, CFGBlockValues &vals,
534 UninitVariablesHandler *handler = 0,
535 bool flagBlockUses = false) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000536
537 if (const BinaryOperator *b = getLogicalOperatorInChain(block)) {
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000538 CFGBlock::const_pred_iterator itr = block->pred_begin();
539 BVPair vA = vals.getBitVectors(*itr, false);
540 ++itr;
541 BVPair vB = vals.getBitVectors(*itr, false);
542
543 BVPair valsAB;
544
545 if (b->getOpcode() == BO_LAnd) {
546 // Merge the 'F' bits from the first and second.
547 vals.mergeIntoScratch(*(vA.second ? vA.second : vA.first), true);
548 vals.mergeIntoScratch(*(vB.second ? vB.second : vB.first), false);
549 valsAB.first = vA.first;
Ted Kremenek2d4bed12011-01-20 21:25:31 +0000550 valsAB.second = &vals.getScratch();
Ted Kremenek13bd4232011-01-20 17:37:17 +0000551 }
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000552 else {
553 // Merge the 'T' bits from the first and second.
554 assert(b->getOpcode() == BO_LOr);
555 vals.mergeIntoScratch(*vA.first, true);
556 vals.mergeIntoScratch(*vB.first, false);
557 valsAB.first = &vals.getScratch();
558 valsAB.second = vA.second ? vA.second : vA.first;
559 }
560 return vals.updateBitVectors(block, valsAB);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000561 }
562
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000563 // Default behavior: merge in values of predecessor blocks.
Ted Kremenek610068c2011-01-15 02:58:47 +0000564 vals.resetScratch();
565 bool isFirst = true;
566 for (CFGBlock::const_pred_iterator I = block->pred_begin(),
567 E = block->pred_end(); I != E; ++I) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000568 vals.mergeIntoScratch(vals.getBitVector(*I, block), isFirst);
Ted Kremenek610068c2011-01-15 02:58:47 +0000569 isFirst = false;
570 }
571 // Apply the transfer function.
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000572 TransferFunctions tf(vals, cfg, ac, handler, flagBlockUses);
Ted Kremenek610068c2011-01-15 02:58:47 +0000573 for (CFGBlock::const_iterator I = block->begin(), E = block->end();
574 I != E; ++I) {
575 if (const CFGStmt *cs = dyn_cast<CFGStmt>(&*I)) {
576 tf.BlockStmt_Visit(cs->getStmt());
577 }
578 }
Ted Kremenek13bd4232011-01-20 17:37:17 +0000579 return vals.updateBitVectorWithScratch(block);
Ted Kremenek610068c2011-01-15 02:58:47 +0000580}
581
582void clang::runUninitializedVariablesAnalysis(const DeclContext &dc,
583 const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000584 AnalysisContext &ac,
Ted Kremenek610068c2011-01-15 02:58:47 +0000585 UninitVariablesHandler &handler) {
586 CFGBlockValues vals(cfg);
587 vals.computeSetOfDeclarations(dc);
588 if (vals.hasNoDeclarations())
589 return;
590 DataflowWorklist worklist(cfg);
591 llvm::BitVector previouslyVisited(cfg.getNumBlockIDs());
592
593 worklist.enqueueSuccessors(&cfg.getEntry());
594
595 while (const CFGBlock *block = worklist.dequeue()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000596 // Did the block change?
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000597 bool changed = runOnBlock(block, cfg, ac, vals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000598 if (changed || !previouslyVisited[block->getBlockID()])
599 worklist.enqueueSuccessors(block);
600 previouslyVisited[block->getBlockID()] = true;
601 }
602
603 // Run through the blocks one more time, and report uninitialized variabes.
604 for (CFG::const_iterator BI = cfg.begin(), BE = cfg.end(); BI != BE; ++BI) {
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000605 runOnBlock(*BI, cfg, ac, vals, &handler, /* flagBlockUses */ true);
Ted Kremenek610068c2011-01-15 02:58:47 +0000606 }
607}
608
609UninitVariablesHandler::~UninitVariablesHandler() {}
610