2011-03-15 11:17:07 +08:00
|
|
|
//==- UninitializedValues.cpp - Find Uninitialized Values -------*- C++ --*-==//
|
2011-01-15 10:58:47 +08:00
|
|
|
//
|
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
//
|
|
|
|
// This file implements uninitialized values analysis for source-level CFGs.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2011-01-21 01:37:17 +08:00
|
|
|
#include <utility>
|
2011-01-15 10:58:47 +08:00
|
|
|
#include "llvm/ADT/Optional.h"
|
2012-09-29 00:44:29 +08:00
|
|
|
#include "llvm/ADT/SmallBitVector.h"
|
2011-01-15 10:58:47 +08:00
|
|
|
#include "llvm/ADT/SmallVector.h"
|
2011-05-31 11:56:09 +08:00
|
|
|
#include "llvm/ADT/PackedVector.h"
|
2011-01-15 10:58:47 +08:00
|
|
|
#include "llvm/ADT/DenseMap.h"
|
2012-07-14 07:33:44 +08:00
|
|
|
#include "clang/AST/ASTContext.h"
|
2011-01-15 10:58:47 +08:00
|
|
|
#include "clang/AST/Decl.h"
|
|
|
|
#include "clang/Analysis/CFG.h"
|
2011-01-26 03:13:48 +08:00
|
|
|
#include "clang/Analysis/AnalysisContext.h"
|
2011-01-15 10:58:47 +08:00
|
|
|
#include "clang/Analysis/Visitors/CFGRecStmtDeclVisitor.h"
|
2011-03-15 11:17:07 +08:00
|
|
|
#include "clang/Analysis/Analyses/UninitializedValues.h"
|
2012-09-13 08:21:35 +08:00
|
|
|
#include "clang/Analysis/DomainSpecific/ObjCNoReturn.h"
|
2012-03-02 03:45:56 +08:00
|
|
|
#include "llvm/Support/SaveAndRestore.h"
|
2011-01-15 10:58:47 +08:00
|
|
|
|
|
|
|
using namespace clang;
|
|
|
|
|
2012-07-14 07:33:44 +08:00
|
|
|
#define DEBUG_LOGGING 0
|
|
|
|
|
2011-01-27 10:29:34 +08:00
|
|
|
static bool isTrackedVar(const VarDecl *vd, const DeclContext *dc) {
|
2011-03-17 11:06:11 +08:00
|
|
|
if (vd->isLocalVarDecl() && !vd->hasGlobalStorage() &&
|
2011-04-08 04:02:56 +08:00
|
|
|
!vd->isExceptionVariable() &&
|
2011-03-17 11:06:11 +08:00
|
|
|
vd->getDeclContext() == dc) {
|
|
|
|
QualType ty = vd->getType();
|
|
|
|
return ty->isScalarType() || ty->isVectorType();
|
|
|
|
}
|
|
|
|
return false;
|
2011-01-18 12:53:25 +08:00
|
|
|
}
|
|
|
|
|
2011-01-15 10:58:47 +08:00
|
|
|
//------------------------------------------------------------------------====//
|
2011-03-15 12:57:27 +08:00
|
|
|
// DeclToIndex: a mapping from Decls we track to value indices.
|
2011-01-15 10:58:47 +08:00
|
|
|
//====------------------------------------------------------------------------//
|
|
|
|
|
|
|
|
namespace {
|
2011-03-15 12:57:27 +08:00
|
|
|
class DeclToIndex {
|
2011-01-15 10:58:47 +08:00
|
|
|
llvm::DenseMap<const VarDecl *, unsigned> map;
|
|
|
|
public:
|
2011-03-15 12:57:27 +08:00
|
|
|
DeclToIndex() {}
|
2011-01-15 10:58:47 +08:00
|
|
|
|
|
|
|
/// Compute the actual mapping from declarations to bits.
|
|
|
|
void computeMap(const DeclContext &dc);
|
|
|
|
|
|
|
|
/// Return the number of declarations in the map.
|
|
|
|
unsigned size() const { return map.size(); }
|
|
|
|
|
|
|
|
/// Returns the bit vector index for a given declaration.
|
2011-03-29 09:40:00 +08:00
|
|
|
llvm::Optional<unsigned> getValueIndex(const VarDecl *d) const;
|
2011-01-15 10:58:47 +08:00
|
|
|
};
|
|
|
|
}
|
|
|
|
|
2011-03-15 12:57:27 +08:00
|
|
|
void DeclToIndex::computeMap(const DeclContext &dc) {
|
2011-01-15 10:58:47 +08:00
|
|
|
unsigned count = 0;
|
|
|
|
DeclContext::specific_decl_iterator<VarDecl> I(dc.decls_begin()),
|
|
|
|
E(dc.decls_end());
|
|
|
|
for ( ; I != E; ++I) {
|
2012-06-07 04:45:41 +08:00
|
|
|
const VarDecl *vd = *I;
|
2011-01-27 10:29:34 +08:00
|
|
|
if (isTrackedVar(vd, &dc))
|
2011-01-15 10:58:47 +08:00
|
|
|
map[vd] = count++;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2011-03-29 09:40:00 +08:00
|
|
|
llvm::Optional<unsigned> DeclToIndex::getValueIndex(const VarDecl *d) const {
|
|
|
|
llvm::DenseMap<const VarDecl *, unsigned>::const_iterator I = map.find(d);
|
2011-01-15 10:58:47 +08:00
|
|
|
if (I == map.end())
|
|
|
|
return llvm::Optional<unsigned>();
|
|
|
|
return I->second;
|
|
|
|
}
|
|
|
|
|
|
|
|
//------------------------------------------------------------------------====//
|
|
|
|
// CFGBlockValues: dataflow values for CFG blocks.
|
|
|
|
//====------------------------------------------------------------------------//
|
|
|
|
|
2011-03-15 12:57:38 +08:00
|
|
|
// These values are defined in such a way that a merge can be done using
|
|
|
|
// a bitwise OR.
|
|
|
|
enum Value { Unknown = 0x0, /* 00 */
|
|
|
|
Initialized = 0x1, /* 01 */
|
|
|
|
Uninitialized = 0x2, /* 10 */
|
|
|
|
MayUninitialized = 0x3 /* 11 */ };
|
|
|
|
|
|
|
|
static bool isUninitialized(const Value v) {
|
|
|
|
return v >= Uninitialized;
|
|
|
|
}
|
|
|
|
static bool isAlwaysUninit(const Value v) {
|
|
|
|
return v == Uninitialized;
|
|
|
|
}
|
2011-03-15 12:57:29 +08:00
|
|
|
|
2011-03-26 20:38:21 +08:00
|
|
|
namespace {
|
2011-03-15 12:57:29 +08:00
|
|
|
|
2012-09-29 00:44:29 +08:00
|
|
|
typedef llvm::PackedVector<Value, 2, llvm::SmallBitVector> ValueVector;
|
2011-01-21 01:37:17 +08:00
|
|
|
|
2011-01-15 10:58:47 +08:00
|
|
|
class CFGBlockValues {
|
|
|
|
const CFG &cfg;
|
2012-09-29 00:44:29 +08:00
|
|
|
SmallVector<ValueVector, 8> vals;
|
2011-03-15 12:57:27 +08:00
|
|
|
ValueVector scratch;
|
2011-03-15 13:30:12 +08:00
|
|
|
DeclToIndex declToIndex;
|
2011-01-15 10:58:47 +08:00
|
|
|
public:
|
|
|
|
CFGBlockValues(const CFG &cfg);
|
2012-07-19 12:59:05 +08:00
|
|
|
|
2011-04-05 07:29:12 +08:00
|
|
|
unsigned getNumEntries() const { return declToIndex.size(); }
|
|
|
|
|
2011-01-15 10:58:47 +08:00
|
|
|
void computeSetOfDeclarations(const DeclContext &dc);
|
2012-07-19 12:59:05 +08:00
|
|
|
ValueVector &getValueVector(const CFGBlock *block) {
|
2012-09-29 00:44:29 +08:00
|
|
|
return vals[block->getBlockID()];
|
2012-07-19 12:59:05 +08:00
|
|
|
}
|
2011-01-21 01:37:17 +08:00
|
|
|
|
2012-07-03 07:23:04 +08:00
|
|
|
void setAllScratchValues(Value V);
|
2011-03-15 12:57:27 +08:00
|
|
|
void mergeIntoScratch(ValueVector const &source, bool isFirst);
|
|
|
|
bool updateValueVectorWithScratch(const CFGBlock *block);
|
2011-01-15 10:58:47 +08:00
|
|
|
|
|
|
|
bool hasNoDeclarations() const {
|
2011-03-15 13:30:12 +08:00
|
|
|
return declToIndex.size() == 0;
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
2011-08-20 09:15:28 +08:00
|
|
|
|
2011-01-15 10:58:47 +08:00
|
|
|
void resetScratch();
|
2011-01-21 01:37:17 +08:00
|
|
|
|
2011-03-15 12:57:27 +08:00
|
|
|
ValueVector::reference operator[](const VarDecl *vd);
|
2012-05-25 10:17:09 +08:00
|
|
|
|
|
|
|
Value getValue(const CFGBlock *block, const CFGBlock *dstBlock,
|
|
|
|
const VarDecl *vd) {
|
|
|
|
const llvm::Optional<unsigned> &idx = declToIndex.getValueIndex(vd);
|
|
|
|
assert(idx.hasValue());
|
2012-07-19 12:59:05 +08:00
|
|
|
return getValueVector(block)[idx.getValue()];
|
2012-05-25 10:17:09 +08:00
|
|
|
}
|
2011-01-15 10:58:47 +08:00
|
|
|
};
|
2011-03-26 20:38:21 +08:00
|
|
|
} // end anonymous namespace
|
2011-01-15 10:58:47 +08:00
|
|
|
|
2012-07-19 12:59:05 +08:00
|
|
|
CFGBlockValues::CFGBlockValues(const CFG &c) : cfg(c), vals(0) {}
|
2011-01-15 10:58:47 +08:00
|
|
|
|
|
|
|
void CFGBlockValues::computeSetOfDeclarations(const DeclContext &dc) {
|
2011-03-15 13:30:12 +08:00
|
|
|
declToIndex.computeMap(dc);
|
2012-07-19 12:59:05 +08:00
|
|
|
unsigned decls = declToIndex.size();
|
|
|
|
scratch.resize(decls);
|
|
|
|
unsigned n = cfg.getNumBlockIDs();
|
|
|
|
if (!n)
|
|
|
|
return;
|
|
|
|
vals.resize(n);
|
|
|
|
for (unsigned i = 0; i < n; ++i)
|
2012-09-29 00:44:29 +08:00
|
|
|
vals[i].resize(decls);
|
2011-01-21 01:37:17 +08:00
|
|
|
}
|
|
|
|
|
2012-07-14 07:33:44 +08:00
|
|
|
#if DEBUG_LOGGING
|
2011-03-15 12:57:27 +08:00
|
|
|
static void printVector(const CFGBlock *block, ValueVector &bv,
|
2011-02-02 01:43:18 +08:00
|
|
|
unsigned num) {
|
|
|
|
llvm::errs() << block->getBlockID() << " :";
|
|
|
|
for (unsigned i = 0; i < bv.size(); ++i) {
|
|
|
|
llvm::errs() << ' ' << bv[i];
|
|
|
|
}
|
|
|
|
llvm::errs() << " : " << num << '\n';
|
|
|
|
}
|
|
|
|
#endif
|
2011-01-15 10:58:47 +08:00
|
|
|
|
2012-07-03 07:23:04 +08:00
|
|
|
void CFGBlockValues::setAllScratchValues(Value V) {
|
|
|
|
for (unsigned I = 0, E = scratch.size(); I != E; ++I)
|
|
|
|
scratch[I] = V;
|
|
|
|
}
|
|
|
|
|
2011-10-07 08:42:48 +08:00
|
|
|
void CFGBlockValues::mergeIntoScratch(ValueVector const &source,
|
|
|
|
bool isFirst) {
|
|
|
|
if (isFirst)
|
|
|
|
scratch = source;
|
|
|
|
else
|
|
|
|
scratch |= source;
|
|
|
|
}
|
|
|
|
|
2011-03-15 12:57:27 +08:00
|
|
|
bool CFGBlockValues::updateValueVectorWithScratch(const CFGBlock *block) {
|
2012-07-19 12:59:05 +08:00
|
|
|
ValueVector &dst = getValueVector(block);
|
2011-01-15 10:58:47 +08:00
|
|
|
bool changed = (dst != scratch);
|
|
|
|
if (changed)
|
|
|
|
dst = scratch;
|
2012-07-14 07:33:44 +08:00
|
|
|
#if DEBUG_LOGGING
|
2011-02-02 01:43:18 +08:00
|
|
|
printVector(block, scratch, 0);
|
|
|
|
#endif
|
2011-01-21 01:37:17 +08:00
|
|
|
return changed;
|
|
|
|
}
|
|
|
|
|
2011-01-15 10:58:47 +08:00
|
|
|
void CFGBlockValues::resetScratch() {
|
|
|
|
scratch.reset();
|
|
|
|
}
|
|
|
|
|
2011-03-15 12:57:27 +08:00
|
|
|
ValueVector::reference CFGBlockValues::operator[](const VarDecl *vd) {
|
2011-03-15 13:30:12 +08:00
|
|
|
const llvm::Optional<unsigned> &idx = declToIndex.getValueIndex(vd);
|
2011-01-15 10:58:47 +08:00
|
|
|
assert(idx.hasValue());
|
|
|
|
return scratch[idx.getValue()];
|
|
|
|
}
|
|
|
|
|
|
|
|
//------------------------------------------------------------------------====//
|
|
|
|
// Worklist: worklist for dataflow analysis.
|
|
|
|
//====------------------------------------------------------------------------//
|
|
|
|
|
|
|
|
namespace {
|
|
|
|
class DataflowWorklist {
|
2011-07-23 18:55:15 +08:00
|
|
|
SmallVector<const CFGBlock *, 20> worklist;
|
2011-03-15 12:57:32 +08:00
|
|
|
llvm::BitVector enqueuedBlocks;
|
2011-01-15 10:58:47 +08:00
|
|
|
public:
|
|
|
|
DataflowWorklist(const CFG &cfg) : enqueuedBlocks(cfg.getNumBlockIDs()) {}
|
|
|
|
|
|
|
|
void enqueueSuccessors(const CFGBlock *block);
|
|
|
|
const CFGBlock *dequeue();
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
|
|
|
void DataflowWorklist::enqueueSuccessors(const clang::CFGBlock *block) {
|
Make the worklist in the uninitialized values checker actually a queue.
Previously, despite the names 'enqueue' and 'dequeue', it behaved as
a stack and visited blocks in a LIFO fashion. This interacts badly with
extremely broad CFGs *inside* of a loop (such as a large switch inside
a state machine) where every block updates a different variable.
When encountering such a CFG, the checker visited blocks in essentially
a "depth first" order due to the stack-like behavior of the work list.
Combined with each block updating a different variable, the saturation
logic of the checker caused it to re-traverse blocks [1,N-1] of the
broad CFG inside the loop after traversing block N. These re-traversals
were to propagate the variable values derived from block N. Assuming
approximately the same number of variables as inner blocks exist, the
end result is O(N^2) updates. By making this a queue, we also make the
traversal essentially "breadth-first" across each of the N inner blocks
of the loop. Then all of this state is propagated around to all N inner
blocks of the loop. The result is O(N) updates.
The truth is in the numbers:
Before, gcc.c: 96409 block visits (max: 61546, avg: 591)
After, gcc.c: 69958 block visits (max: 33090, avg: 429)
Before, PR10183: 2540494 block vists (max: 2536495, avg: 37360)
After, PR10183: 137803 block visits (max: 134406, avg: 2026)
The nearly 20x reduction in work for PR10183 corresponds to a roughly
100x speedup in compile time.
I've tested it on all the code I can get my hands on, and I've seen no
slowdowns due to this change. Where I've collected stats, the ammount of
work done is on average less. I'll also commit shortly some synthetic
test cases useful in analyzing the performance of CFG-based warnings.
Submitting this based on Doug's feedback that post-commit review should
be good. Ted, please review! Hopefully this helps compile times until
then.
llvm-svn: 134697
2011-07-08 19:19:06 +08:00
|
|
|
unsigned OldWorklistSize = worklist.size();
|
2011-01-15 10:58:47 +08:00
|
|
|
for (CFGBlock::const_succ_iterator I = block->succ_begin(),
|
|
|
|
E = block->succ_end(); I != E; ++I) {
|
Make the worklist in the uninitialized values checker actually a queue.
Previously, despite the names 'enqueue' and 'dequeue', it behaved as
a stack and visited blocks in a LIFO fashion. This interacts badly with
extremely broad CFGs *inside* of a loop (such as a large switch inside
a state machine) where every block updates a different variable.
When encountering such a CFG, the checker visited blocks in essentially
a "depth first" order due to the stack-like behavior of the work list.
Combined with each block updating a different variable, the saturation
logic of the checker caused it to re-traverse blocks [1,N-1] of the
broad CFG inside the loop after traversing block N. These re-traversals
were to propagate the variable values derived from block N. Assuming
approximately the same number of variables as inner blocks exist, the
end result is O(N^2) updates. By making this a queue, we also make the
traversal essentially "breadth-first" across each of the N inner blocks
of the loop. Then all of this state is propagated around to all N inner
blocks of the loop. The result is O(N) updates.
The truth is in the numbers:
Before, gcc.c: 96409 block visits (max: 61546, avg: 591)
After, gcc.c: 69958 block visits (max: 33090, avg: 429)
Before, PR10183: 2540494 block vists (max: 2536495, avg: 37360)
After, PR10183: 137803 block visits (max: 134406, avg: 2026)
The nearly 20x reduction in work for PR10183 corresponds to a roughly
100x speedup in compile time.
I've tested it on all the code I can get my hands on, and I've seen no
slowdowns due to this change. Where I've collected stats, the ammount of
work done is on average less. I'll also commit shortly some synthetic
test cases useful in analyzing the performance of CFG-based warnings.
Submitting this based on Doug's feedback that post-commit review should
be good. Ted, please review! Hopefully this helps compile times until
then.
llvm-svn: 134697
2011-07-08 19:19:06 +08:00
|
|
|
const CFGBlock *Successor = *I;
|
|
|
|
if (!Successor || enqueuedBlocks[Successor->getBlockID()])
|
|
|
|
continue;
|
|
|
|
worklist.push_back(Successor);
|
|
|
|
enqueuedBlocks[Successor->getBlockID()] = true;
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
Make the worklist in the uninitialized values checker actually a queue.
Previously, despite the names 'enqueue' and 'dequeue', it behaved as
a stack and visited blocks in a LIFO fashion. This interacts badly with
extremely broad CFGs *inside* of a loop (such as a large switch inside
a state machine) where every block updates a different variable.
When encountering such a CFG, the checker visited blocks in essentially
a "depth first" order due to the stack-like behavior of the work list.
Combined with each block updating a different variable, the saturation
logic of the checker caused it to re-traverse blocks [1,N-1] of the
broad CFG inside the loop after traversing block N. These re-traversals
were to propagate the variable values derived from block N. Assuming
approximately the same number of variables as inner blocks exist, the
end result is O(N^2) updates. By making this a queue, we also make the
traversal essentially "breadth-first" across each of the N inner blocks
of the loop. Then all of this state is propagated around to all N inner
blocks of the loop. The result is O(N) updates.
The truth is in the numbers:
Before, gcc.c: 96409 block visits (max: 61546, avg: 591)
After, gcc.c: 69958 block visits (max: 33090, avg: 429)
Before, PR10183: 2540494 block vists (max: 2536495, avg: 37360)
After, PR10183: 137803 block visits (max: 134406, avg: 2026)
The nearly 20x reduction in work for PR10183 corresponds to a roughly
100x speedup in compile time.
I've tested it on all the code I can get my hands on, and I've seen no
slowdowns due to this change. Where I've collected stats, the ammount of
work done is on average less. I'll also commit shortly some synthetic
test cases useful in analyzing the performance of CFG-based warnings.
Submitting this based on Doug's feedback that post-commit review should
be good. Ted, please review! Hopefully this helps compile times until
then.
llvm-svn: 134697
2011-07-08 19:19:06 +08:00
|
|
|
if (OldWorklistSize == 0 || OldWorklistSize == worklist.size())
|
|
|
|
return;
|
|
|
|
|
|
|
|
// Rotate the newly added blocks to the start of the worklist so that it forms
|
|
|
|
// a proper queue when we pop off the end of the worklist.
|
|
|
|
std::rotate(worklist.begin(), worklist.begin() + OldWorklistSize,
|
|
|
|
worklist.end());
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
const CFGBlock *DataflowWorklist::dequeue() {
|
|
|
|
if (worklist.empty())
|
|
|
|
return 0;
|
|
|
|
const CFGBlock *b = worklist.back();
|
|
|
|
worklist.pop_back();
|
|
|
|
enqueuedBlocks[b->getBlockID()] = false;
|
|
|
|
return b;
|
|
|
|
}
|
|
|
|
|
|
|
|
//------------------------------------------------------------------------====//
|
2012-07-17 08:06:14 +08:00
|
|
|
// Classification of DeclRefExprs as use or initialization.
|
2011-01-15 10:58:47 +08:00
|
|
|
//====------------------------------------------------------------------------//
|
|
|
|
|
|
|
|
namespace {
|
|
|
|
class FindVarResult {
|
|
|
|
const VarDecl *vd;
|
|
|
|
const DeclRefExpr *dr;
|
|
|
|
public:
|
2012-07-17 08:06:14 +08:00
|
|
|
FindVarResult(const VarDecl *vd, const DeclRefExpr *dr) : vd(vd), dr(dr) {}
|
|
|
|
|
2011-01-15 10:58:47 +08:00
|
|
|
const DeclRefExpr *getDeclRefExpr() const { return dr; }
|
|
|
|
const VarDecl *getDecl() const { return vd; }
|
|
|
|
};
|
2012-07-17 08:06:14 +08:00
|
|
|
|
|
|
|
static const Expr *stripCasts(ASTContext &C, const Expr *Ex) {
|
|
|
|
while (Ex) {
|
|
|
|
Ex = Ex->IgnoreParenNoopCasts(C);
|
|
|
|
if (const CastExpr *CE = dyn_cast<CastExpr>(Ex)) {
|
|
|
|
if (CE->getCastKind() == CK_LValueBitCast) {
|
|
|
|
Ex = CE->getSubExpr();
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
return Ex;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// If E is an expression comprising a reference to a single variable, find that
|
|
|
|
/// variable.
|
|
|
|
static FindVarResult findVar(const Expr *E, const DeclContext *DC) {
|
|
|
|
if (const DeclRefExpr *DRE =
|
|
|
|
dyn_cast<DeclRefExpr>(stripCasts(DC->getParentASTContext(), E)))
|
|
|
|
if (const VarDecl *VD = dyn_cast<VarDecl>(DRE->getDecl()))
|
|
|
|
if (isTrackedVar(VD, DC))
|
|
|
|
return FindVarResult(VD, DRE);
|
|
|
|
return FindVarResult(0, 0);
|
|
|
|
}
|
|
|
|
|
|
|
|
/// \brief Classify each DeclRefExpr as an initialization or a use. Any
|
|
|
|
/// DeclRefExpr which isn't explicitly classified will be assumed to have
|
|
|
|
/// escaped the analysis and will be treated as an initialization.
|
|
|
|
class ClassifyRefs : public StmtVisitor<ClassifyRefs> {
|
|
|
|
public:
|
|
|
|
enum Class {
|
|
|
|
Init,
|
|
|
|
Use,
|
|
|
|
SelfInit,
|
|
|
|
Ignore
|
|
|
|
};
|
|
|
|
|
|
|
|
private:
|
|
|
|
const DeclContext *DC;
|
|
|
|
llvm::DenseMap<const DeclRefExpr*, Class> Classification;
|
|
|
|
|
|
|
|
bool isTrackedVar(const VarDecl *VD) const {
|
|
|
|
return ::isTrackedVar(VD, DC);
|
|
|
|
}
|
|
|
|
|
|
|
|
void classify(const Expr *E, Class C);
|
|
|
|
|
|
|
|
public:
|
|
|
|
ClassifyRefs(AnalysisDeclContext &AC) : DC(cast<DeclContext>(AC.getDecl())) {}
|
|
|
|
|
|
|
|
void VisitDeclStmt(DeclStmt *DS);
|
|
|
|
void VisitUnaryOperator(UnaryOperator *UO);
|
|
|
|
void VisitBinaryOperator(BinaryOperator *BO);
|
|
|
|
void VisitCallExpr(CallExpr *CE);
|
|
|
|
void VisitCastExpr(CastExpr *CE);
|
|
|
|
|
|
|
|
void operator()(Stmt *S) { Visit(S); }
|
|
|
|
|
|
|
|
Class get(const DeclRefExpr *DRE) const {
|
|
|
|
llvm::DenseMap<const DeclRefExpr*, Class>::const_iterator I
|
|
|
|
= Classification.find(DRE);
|
|
|
|
if (I != Classification.end())
|
|
|
|
return I->second;
|
|
|
|
|
|
|
|
const VarDecl *VD = dyn_cast<VarDecl>(DRE->getDecl());
|
|
|
|
if (!VD || !isTrackedVar(VD))
|
|
|
|
return Ignore;
|
|
|
|
|
|
|
|
return Init;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
|
|
|
static const DeclRefExpr *getSelfInitExpr(VarDecl *VD) {
|
|
|
|
if (Expr *Init = VD->getInit()) {
|
|
|
|
const DeclRefExpr *DRE
|
|
|
|
= dyn_cast<DeclRefExpr>(stripCasts(VD->getASTContext(), Init));
|
|
|
|
if (DRE && DRE->getDecl() == VD)
|
|
|
|
return DRE;
|
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
void ClassifyRefs::classify(const Expr *E, Class C) {
|
|
|
|
FindVarResult Var = findVar(E, DC);
|
|
|
|
if (const DeclRefExpr *DRE = Var.getDeclRefExpr())
|
|
|
|
Classification[DRE] = std::max(Classification[DRE], C);
|
|
|
|
}
|
|
|
|
|
|
|
|
void ClassifyRefs::VisitDeclStmt(DeclStmt *DS) {
|
|
|
|
for (DeclStmt::decl_iterator DI = DS->decl_begin(), DE = DS->decl_end();
|
|
|
|
DI != DE; ++DI) {
|
|
|
|
VarDecl *VD = dyn_cast<VarDecl>(*DI);
|
|
|
|
if (VD && isTrackedVar(VD))
|
|
|
|
if (const DeclRefExpr *DRE = getSelfInitExpr(VD))
|
|
|
|
Classification[DRE] = SelfInit;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
void ClassifyRefs::VisitBinaryOperator(BinaryOperator *BO) {
|
|
|
|
// Ignore the evaluation of a DeclRefExpr on the LHS of an assignment. If this
|
|
|
|
// is not a compound-assignment, we will treat it as initializing the variable
|
|
|
|
// when TransferFunctions visits it. A compound-assignment does not affect
|
|
|
|
// whether a variable is uninitialized, and there's no point counting it as a
|
|
|
|
// use.
|
2012-07-17 09:27:33 +08:00
|
|
|
if (BO->isCompoundAssignmentOp())
|
|
|
|
classify(BO->getLHS(), Use);
|
|
|
|
else if (BO->getOpcode() == BO_Assign)
|
2012-07-17 08:06:14 +08:00
|
|
|
classify(BO->getLHS(), Ignore);
|
|
|
|
}
|
|
|
|
|
|
|
|
void ClassifyRefs::VisitUnaryOperator(UnaryOperator *UO) {
|
|
|
|
// Increment and decrement are uses despite there being no lvalue-to-rvalue
|
|
|
|
// conversion.
|
|
|
|
if (UO->isIncrementDecrementOp())
|
|
|
|
classify(UO->getSubExpr(), Use);
|
|
|
|
}
|
|
|
|
|
|
|
|
void ClassifyRefs::VisitCallExpr(CallExpr *CE) {
|
|
|
|
// If a value is passed by const reference to a function, we should not assume
|
|
|
|
// that it is initialized by the call, and we conservatively do not assume
|
|
|
|
// that it is used.
|
|
|
|
for (CallExpr::arg_iterator I = CE->arg_begin(), E = CE->arg_end();
|
|
|
|
I != E; ++I)
|
|
|
|
if ((*I)->getType().isConstQualified() && (*I)->isGLValue())
|
|
|
|
classify(*I, Ignore);
|
|
|
|
}
|
|
|
|
|
|
|
|
void ClassifyRefs::VisitCastExpr(CastExpr *CE) {
|
|
|
|
if (CE->getCastKind() == CK_LValueToRValue)
|
|
|
|
classify(CE->getSubExpr(), Use);
|
|
|
|
else if (CStyleCastExpr *CSE = dyn_cast<CStyleCastExpr>(CE)) {
|
|
|
|
if (CSE->getType()->isVoidType()) {
|
|
|
|
// Squelch any detected load of an uninitialized value if
|
|
|
|
// we cast it to void.
|
|
|
|
// e.g. (void) x;
|
|
|
|
classify(CSE->getSubExpr(), Ignore);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
//------------------------------------------------------------------------====//
|
|
|
|
// Transfer function for uninitialized values analysis.
|
|
|
|
//====------------------------------------------------------------------------//
|
|
|
|
|
|
|
|
namespace {
|
2011-07-19 22:18:48 +08:00
|
|
|
class TransferFunctions : public StmtVisitor<TransferFunctions> {
|
2011-01-15 10:58:47 +08:00
|
|
|
CFGBlockValues &vals;
|
|
|
|
const CFG &cfg;
|
2012-05-25 10:17:09 +08:00
|
|
|
const CFGBlock *block;
|
2011-10-24 09:32:45 +08:00
|
|
|
AnalysisDeclContext ∾
|
2012-07-17 08:06:14 +08:00
|
|
|
const ClassifyRefs &classification;
|
2012-09-13 08:21:35 +08:00
|
|
|
ObjCNoReturn objCNoRet;
|
2011-01-15 10:58:47 +08:00
|
|
|
UninitVariablesHandler *handler;
|
2012-07-17 08:06:14 +08:00
|
|
|
|
2011-01-15 10:58:47 +08:00
|
|
|
public:
|
|
|
|
TransferFunctions(CFGBlockValues &vals, const CFG &cfg,
|
2012-05-25 10:17:09 +08:00
|
|
|
const CFGBlock *block, AnalysisDeclContext &ac,
|
2012-07-17 08:06:14 +08:00
|
|
|
const ClassifyRefs &classification,
|
2011-09-03 03:39:26 +08:00
|
|
|
UninitVariablesHandler *handler)
|
2012-07-17 08:06:14 +08:00
|
|
|
: vals(vals), cfg(cfg), block(block), ac(ac),
|
2012-09-13 08:21:35 +08:00
|
|
|
classification(classification), objCNoRet(ac.getASTContext()),
|
|
|
|
handler(handler) {}
|
2012-07-17 08:06:14 +08:00
|
|
|
|
2012-05-25 07:45:35 +08:00
|
|
|
void reportUse(const Expr *ex, const VarDecl *vd);
|
2011-01-26 03:13:48 +08:00
|
|
|
|
2012-09-13 08:21:35 +08:00
|
|
|
void VisitBinaryOperator(BinaryOperator *bo);
|
2011-01-26 03:13:48 +08:00
|
|
|
void VisitBlockExpr(BlockExpr *be);
|
2012-07-03 07:23:04 +08:00
|
|
|
void VisitCallExpr(CallExpr *ce);
|
2011-01-19 05:18:58 +08:00
|
|
|
void VisitDeclRefExpr(DeclRefExpr *dr);
|
2012-09-13 08:21:35 +08:00
|
|
|
void VisitDeclStmt(DeclStmt *ds);
|
|
|
|
void VisitObjCForCollectionStmt(ObjCForCollectionStmt *FS);
|
|
|
|
void VisitObjCMessageExpr(ObjCMessageExpr *ME);
|
2012-05-25 10:17:09 +08:00
|
|
|
|
2011-01-27 10:29:34 +08:00
|
|
|
bool isTrackedVar(const VarDecl *vd) {
|
|
|
|
return ::isTrackedVar(vd, cast<DeclContext>(ac.getDecl()));
|
|
|
|
}
|
2012-05-25 10:17:09 +08:00
|
|
|
|
2012-07-17 08:06:14 +08:00
|
|
|
FindVarResult findVar(const Expr *ex) {
|
|
|
|
return ::findVar(ex, cast<DeclContext>(ac.getDecl()));
|
|
|
|
}
|
|
|
|
|
2012-05-25 10:17:09 +08:00
|
|
|
UninitUse getUninitUse(const Expr *ex, const VarDecl *vd, Value v) {
|
|
|
|
UninitUse Use(ex, isAlwaysUninit(v));
|
|
|
|
|
|
|
|
assert(isUninitialized(v));
|
|
|
|
if (Use.getKind() == UninitUse::Always)
|
|
|
|
return Use;
|
|
|
|
|
|
|
|
// If an edge which leads unconditionally to this use did not initialize
|
|
|
|
// the variable, we can say something stronger than 'may be uninitialized':
|
|
|
|
// we can say 'either it's used uninitialized or you have dead code'.
|
|
|
|
//
|
|
|
|
// We track the number of successors of a node which have been visited, and
|
|
|
|
// visit a node once we have visited all of its successors. Only edges where
|
|
|
|
// the variable might still be uninitialized are followed. Since a variable
|
|
|
|
// can't transfer from being initialized to being uninitialized, this will
|
|
|
|
// trace out the subgraph which inevitably leads to the use and does not
|
|
|
|
// initialize the variable. We do not want to skip past loops, since their
|
|
|
|
// non-termination might be correlated with the initialization condition.
|
|
|
|
//
|
|
|
|
// For example:
|
|
|
|
//
|
|
|
|
// void f(bool a, bool b) {
|
|
|
|
// block1: int n;
|
|
|
|
// if (a) {
|
|
|
|
// block2: if (b)
|
|
|
|
// block3: n = 1;
|
|
|
|
// block4: } else if (b) {
|
|
|
|
// block5: while (!a) {
|
|
|
|
// block6: do_work(&a);
|
|
|
|
// n = 2;
|
|
|
|
// }
|
|
|
|
// }
|
|
|
|
// block7: if (a)
|
|
|
|
// block8: g();
|
|
|
|
// block9: return n;
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// Starting from the maybe-uninitialized use in block 9:
|
|
|
|
// * Block 7 is not visited because we have only visited one of its two
|
|
|
|
// successors.
|
|
|
|
// * Block 8 is visited because we've visited its only successor.
|
|
|
|
// From block 8:
|
|
|
|
// * Block 7 is visited because we've now visited both of its successors.
|
|
|
|
// From block 7:
|
|
|
|
// * Blocks 1, 2, 4, 5, and 6 are not visited because we didn't visit all
|
|
|
|
// of their successors (we didn't visit 4, 3, 5, 6, and 5, respectively).
|
|
|
|
// * Block 3 is not visited because it initializes 'n'.
|
|
|
|
// Now the algorithm terminates, having visited blocks 7 and 8, and having
|
|
|
|
// found the frontier is blocks 2, 4, and 5.
|
|
|
|
//
|
|
|
|
// 'n' is definitely uninitialized for two edges into block 7 (from blocks 2
|
|
|
|
// and 4), so we report that any time either of those edges is taken (in
|
|
|
|
// each case when 'b == false'), 'n' is used uninitialized.
|
|
|
|
llvm::SmallVector<const CFGBlock*, 32> Queue;
|
|
|
|
llvm::SmallVector<unsigned, 32> SuccsVisited(cfg.getNumBlockIDs(), 0);
|
|
|
|
Queue.push_back(block);
|
|
|
|
// Specify that we've already visited all successors of the starting block.
|
|
|
|
// This has the dual purpose of ensuring we never add it to the queue, and
|
|
|
|
// of marking it as not being a candidate element of the frontier.
|
|
|
|
SuccsVisited[block->getBlockID()] = block->succ_size();
|
|
|
|
while (!Queue.empty()) {
|
|
|
|
const CFGBlock *B = Queue.back();
|
|
|
|
Queue.pop_back();
|
|
|
|
for (CFGBlock::const_pred_iterator I = B->pred_begin(), E = B->pred_end();
|
|
|
|
I != E; ++I) {
|
|
|
|
const CFGBlock *Pred = *I;
|
|
|
|
if (vals.getValue(Pred, B, vd) == Initialized)
|
|
|
|
// This block initializes the variable.
|
|
|
|
continue;
|
|
|
|
|
2012-07-14 07:33:44 +08:00
|
|
|
unsigned &SV = SuccsVisited[Pred->getBlockID()];
|
|
|
|
if (!SV) {
|
|
|
|
// When visiting the first successor of a block, mark all NULL
|
|
|
|
// successors as having been visited.
|
|
|
|
for (CFGBlock::const_succ_iterator SI = Pred->succ_begin(),
|
|
|
|
SE = Pred->succ_end();
|
|
|
|
SI != SE; ++SI)
|
|
|
|
if (!*SI)
|
|
|
|
++SV;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (++SV == Pred->succ_size())
|
2012-05-25 10:17:09 +08:00
|
|
|
// All paths from this block lead to the use and don't initialize the
|
|
|
|
// variable.
|
|
|
|
Queue.push_back(Pred);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// Scan the frontier, looking for blocks where the variable was
|
|
|
|
// uninitialized.
|
|
|
|
for (CFG::const_iterator BI = cfg.begin(), BE = cfg.end(); BI != BE; ++BI) {
|
|
|
|
const CFGBlock *Block = *BI;
|
|
|
|
unsigned BlockID = Block->getBlockID();
|
|
|
|
const Stmt *Term = Block->getTerminator();
|
|
|
|
if (SuccsVisited[BlockID] && SuccsVisited[BlockID] < Block->succ_size() &&
|
|
|
|
Term) {
|
|
|
|
// This block inevitably leads to the use. If we have an edge from here
|
|
|
|
// to a post-dominator block, and the variable is uninitialized on that
|
|
|
|
// edge, we have found a bug.
|
|
|
|
for (CFGBlock::const_succ_iterator I = Block->succ_begin(),
|
|
|
|
E = Block->succ_end(); I != E; ++I) {
|
|
|
|
const CFGBlock *Succ = *I;
|
|
|
|
if (Succ && SuccsVisited[Succ->getBlockID()] >= Succ->succ_size() &&
|
|
|
|
vals.getValue(Block, Succ, vd) == Uninitialized) {
|
|
|
|
// Switch cases are a special case: report the label to the caller
|
|
|
|
// as the 'terminator', not the switch statement itself. Suppress
|
|
|
|
// situations where no label matched: we can't be sure that's
|
|
|
|
// possible.
|
|
|
|
if (isa<SwitchStmt>(Term)) {
|
|
|
|
const Stmt *Label = Succ->getLabel();
|
|
|
|
if (!Label || !isa<SwitchCase>(Label))
|
|
|
|
// Might not be possible.
|
|
|
|
continue;
|
|
|
|
UninitUse::Branch Branch;
|
|
|
|
Branch.Terminator = Label;
|
|
|
|
Branch.Output = 0; // Ignored.
|
|
|
|
Use.addUninitBranch(Branch);
|
|
|
|
} else {
|
|
|
|
UninitUse::Branch Branch;
|
|
|
|
Branch.Terminator = Term;
|
|
|
|
Branch.Output = I - Block->succ_begin();
|
|
|
|
Use.addUninitBranch(Branch);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return Use;
|
|
|
|
}
|
2011-01-15 10:58:47 +08:00
|
|
|
};
|
|
|
|
}
|
|
|
|
|
2012-05-25 07:45:35 +08:00
|
|
|
void TransferFunctions::reportUse(const Expr *ex, const VarDecl *vd) {
|
|
|
|
if (!handler)
|
|
|
|
return;
|
|
|
|
Value v = vals[vd];
|
|
|
|
if (isUninitialized(v))
|
2012-05-25 10:17:09 +08:00
|
|
|
handler->handleUseOfUninitVariable(vd, getUninitUse(ex, vd, v));
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
|
2012-07-17 08:06:14 +08:00
|
|
|
void TransferFunctions::VisitObjCForCollectionStmt(ObjCForCollectionStmt *FS) {
|
2011-01-27 10:01:31 +08:00
|
|
|
// This represents an initialization of the 'element' value.
|
2012-07-17 08:06:14 +08:00
|
|
|
if (DeclStmt *DS = dyn_cast<DeclStmt>(FS->getElement())) {
|
|
|
|
const VarDecl *VD = cast<VarDecl>(DS->getSingleDecl());
|
|
|
|
if (isTrackedVar(VD))
|
|
|
|
vals[VD] = Initialized;
|
2011-01-27 10:01:31 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2011-01-26 03:13:48 +08:00
|
|
|
void TransferFunctions::VisitBlockExpr(BlockExpr *be) {
|
2011-04-01 06:32:41 +08:00
|
|
|
const BlockDecl *bd = be->getBlockDecl();
|
|
|
|
for (BlockDecl::capture_const_iterator i = bd->capture_begin(),
|
|
|
|
e = bd->capture_end() ; i != e; ++i) {
|
|
|
|
const VarDecl *vd = i->getVariable();
|
|
|
|
if (!isTrackedVar(vd))
|
|
|
|
continue;
|
|
|
|
if (i->isByRef()) {
|
|
|
|
vals[vd] = Initialized;
|
|
|
|
continue;
|
|
|
|
}
|
2012-05-25 07:45:35 +08:00
|
|
|
reportUse(be, vd);
|
2011-01-26 03:13:48 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-07-03 07:23:04 +08:00
|
|
|
void TransferFunctions::VisitCallExpr(CallExpr *ce) {
|
2012-09-12 13:53:43 +08:00
|
|
|
if (Decl *Callee = ce->getCalleeDecl()) {
|
|
|
|
if (Callee->hasAttr<ReturnsTwiceAttr>()) {
|
|
|
|
// After a call to a function like setjmp or vfork, any variable which is
|
|
|
|
// initialized anywhere within this function may now be initialized. For
|
|
|
|
// now, just assume such a call initializes all variables. FIXME: Only
|
|
|
|
// mark variables as initialized if they have an initializer which is
|
|
|
|
// reachable from here.
|
|
|
|
vals.setAllScratchValues(Initialized);
|
|
|
|
}
|
|
|
|
else if (Callee->hasAttr<AnalyzerNoReturnAttr>()) {
|
|
|
|
// Functions labeled like "analyzer_noreturn" are often used to denote
|
|
|
|
// "panic" functions that in special debug situations can still return,
|
|
|
|
// but for the most part should not be treated as returning. This is a
|
|
|
|
// useful annotation borrowed from the static analyzer that is useful for
|
|
|
|
// suppressing branch-specific false positives when we call one of these
|
|
|
|
// functions but keep pretending the path continues (when in reality the
|
|
|
|
// user doesn't care).
|
|
|
|
vals.setAllScratchValues(Unknown);
|
|
|
|
}
|
|
|
|
}
|
2012-07-03 07:23:04 +08:00
|
|
|
}
|
|
|
|
|
2011-07-19 22:18:48 +08:00
|
|
|
void TransferFunctions::VisitDeclRefExpr(DeclRefExpr *dr) {
|
2012-07-17 08:06:14 +08:00
|
|
|
switch (classification.get(dr)) {
|
|
|
|
case ClassifyRefs::Ignore:
|
|
|
|
break;
|
|
|
|
case ClassifyRefs::Use:
|
|
|
|
reportUse(dr, cast<VarDecl>(dr->getDecl()));
|
|
|
|
break;
|
|
|
|
case ClassifyRefs::Init:
|
|
|
|
vals[cast<VarDecl>(dr->getDecl())] = Initialized;
|
|
|
|
break;
|
|
|
|
case ClassifyRefs::SelfInit:
|
|
|
|
if (handler)
|
|
|
|
handler->handleSelfInit(cast<VarDecl>(dr->getDecl()));
|
|
|
|
break;
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-07-17 08:06:14 +08:00
|
|
|
void TransferFunctions::VisitBinaryOperator(BinaryOperator *BO) {
|
|
|
|
if (BO->getOpcode() == BO_Assign) {
|
|
|
|
FindVarResult Var = findVar(BO->getLHS());
|
|
|
|
if (const VarDecl *VD = Var.getDecl())
|
|
|
|
vals[VD] = Initialized;
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-07-17 08:06:14 +08:00
|
|
|
void TransferFunctions::VisitDeclStmt(DeclStmt *DS) {
|
|
|
|
for (DeclStmt::decl_iterator DI = DS->decl_begin(), DE = DS->decl_end();
|
|
|
|
DI != DE; ++DI) {
|
|
|
|
VarDecl *VD = dyn_cast<VarDecl>(*DI);
|
|
|
|
if (VD && isTrackedVar(VD)) {
|
|
|
|
if (getSelfInitExpr(VD)) {
|
|
|
|
// If the initializer consists solely of a reference to itself, we
|
|
|
|
// explicitly mark the variable as uninitialized. This allows code
|
|
|
|
// like the following:
|
|
|
|
//
|
|
|
|
// int x = x;
|
|
|
|
//
|
|
|
|
// to deliberately leave a variable uninitialized. Different analysis
|
|
|
|
// clients can detect this pattern and adjust their reporting
|
|
|
|
// appropriately, but we need to continue to analyze subsequent uses
|
|
|
|
// of the variable.
|
|
|
|
vals[VD] = Uninitialized;
|
|
|
|
} else if (VD->getInit()) {
|
|
|
|
// Treat the new variable as initialized.
|
|
|
|
vals[VD] = Initialized;
|
|
|
|
} else {
|
|
|
|
// No initializer: the variable is now uninitialized. This matters
|
|
|
|
// for cases like:
|
|
|
|
// while (...) {
|
|
|
|
// int n;
|
|
|
|
// use(n);
|
|
|
|
// n = 0;
|
|
|
|
// }
|
|
|
|
// FIXME: Mark the variable as uninitialized whenever its scope is
|
|
|
|
// left, since its scope could be re-entered by a jump over the
|
|
|
|
// declaration.
|
|
|
|
vals[VD] = Uninitialized;
|
2011-07-19 22:18:48 +08:00
|
|
|
}
|
2011-01-26 12:49:43 +08:00
|
|
|
}
|
|
|
|
}
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
|
2012-09-13 08:21:35 +08:00
|
|
|
void TransferFunctions::VisitObjCMessageExpr(ObjCMessageExpr *ME) {
|
|
|
|
// If the Objective-C message expression is an implicit no-return that
|
|
|
|
// is not modeled in the CFG, set the tracked dataflow values to Unknown.
|
|
|
|
if (objCNoRet.isImplicitNoReturn(ME)) {
|
|
|
|
vals.setAllScratchValues(Unknown);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2011-01-15 10:58:47 +08:00
|
|
|
//------------------------------------------------------------------------====//
|
|
|
|
// High-level "driver" logic for uninitialized values analysis.
|
|
|
|
//====------------------------------------------------------------------------//
|
|
|
|
|
2011-01-21 01:37:17 +08:00
|
|
|
static bool runOnBlock(const CFGBlock *block, const CFG &cfg,
|
2011-10-24 09:32:45 +08:00
|
|
|
AnalysisDeclContext &ac, CFGBlockValues &vals,
|
2012-07-17 08:06:14 +08:00
|
|
|
const ClassifyRefs &classification,
|
2011-04-05 04:30:58 +08:00
|
|
|
llvm::BitVector &wasAnalyzed,
|
2011-09-03 03:39:26 +08:00
|
|
|
UninitVariablesHandler *handler = 0) {
|
2011-04-05 04:30:58 +08:00
|
|
|
wasAnalyzed[block->getBlockID()] = true;
|
2011-01-15 10:58:47 +08:00
|
|
|
vals.resetScratch();
|
2012-07-19 12:59:05 +08:00
|
|
|
// Merge in values of predecessor blocks.
|
2011-01-15 10:58:47 +08:00
|
|
|
bool isFirst = true;
|
|
|
|
for (CFGBlock::const_pred_iterator I = block->pred_begin(),
|
|
|
|
E = block->pred_end(); I != E; ++I) {
|
2011-09-03 03:39:26 +08:00
|
|
|
const CFGBlock *pred = *I;
|
|
|
|
if (wasAnalyzed[pred->getBlockID()]) {
|
2012-07-19 12:59:05 +08:00
|
|
|
vals.mergeIntoScratch(vals.getValueVector(pred), isFirst);
|
2011-09-03 03:39:26 +08:00
|
|
|
isFirst = false;
|
|
|
|
}
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
// Apply the transfer function.
|
2012-07-17 08:06:14 +08:00
|
|
|
TransferFunctions tf(vals, cfg, block, ac, classification, handler);
|
2011-01-15 10:58:47 +08:00
|
|
|
for (CFGBlock::const_iterator I = block->begin(), E = block->end();
|
|
|
|
I != E; ++I) {
|
|
|
|
if (const CFGStmt *cs = dyn_cast<CFGStmt>(&*I)) {
|
2011-08-24 07:05:04 +08:00
|
|
|
tf.Visit(const_cast<Stmt*>(cs->getStmt()));
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
}
|
2011-03-15 12:57:27 +08:00
|
|
|
return vals.updateValueVectorWithScratch(block);
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
|
2011-07-07 00:21:37 +08:00
|
|
|
void clang::runUninitializedVariablesAnalysis(
|
|
|
|
const DeclContext &dc,
|
|
|
|
const CFG &cfg,
|
2011-10-24 09:32:45 +08:00
|
|
|
AnalysisDeclContext &ac,
|
2011-07-07 00:21:37 +08:00
|
|
|
UninitVariablesHandler &handler,
|
|
|
|
UninitVariablesAnalysisStats &stats) {
|
2011-01-15 10:58:47 +08:00
|
|
|
CFGBlockValues vals(cfg);
|
|
|
|
vals.computeSetOfDeclarations(dc);
|
|
|
|
if (vals.hasNoDeclarations())
|
|
|
|
return;
|
2011-04-05 07:29:12 +08:00
|
|
|
|
2011-07-07 00:21:37 +08:00
|
|
|
stats.NumVariablesAnalyzed = vals.getNumEntries();
|
|
|
|
|
2012-07-17 08:06:14 +08:00
|
|
|
// Precompute which expressions are uses and which are initializations.
|
|
|
|
ClassifyRefs classification(ac);
|
|
|
|
cfg.VisitBlockStmts(classification);
|
|
|
|
|
2011-04-05 07:29:12 +08:00
|
|
|
// Mark all variables uninitialized at the entry.
|
|
|
|
const CFGBlock &entry = cfg.getEntry();
|
2012-07-19 12:59:05 +08:00
|
|
|
ValueVector &vec = vals.getValueVector(&entry);
|
|
|
|
const unsigned n = vals.getNumEntries();
|
|
|
|
for (unsigned j = 0; j < n ; ++j) {
|
|
|
|
vec[j] = Uninitialized;
|
2011-04-05 07:29:12 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// Proceed with the workist.
|
2011-01-15 10:58:47 +08:00
|
|
|
DataflowWorklist worklist(cfg);
|
2011-03-15 12:57:32 +08:00
|
|
|
llvm::BitVector previouslyVisited(cfg.getNumBlockIDs());
|
2011-01-15 10:58:47 +08:00
|
|
|
worklist.enqueueSuccessors(&cfg.getEntry());
|
2011-04-05 04:30:58 +08:00
|
|
|
llvm::BitVector wasAnalyzed(cfg.getNumBlockIDs(), false);
|
2011-09-03 03:39:26 +08:00
|
|
|
wasAnalyzed[cfg.getEntry().getBlockID()] = true;
|
2011-01-15 10:58:47 +08:00
|
|
|
|
|
|
|
while (const CFGBlock *block = worklist.dequeue()) {
|
|
|
|
// Did the block change?
|
2012-07-17 08:06:14 +08:00
|
|
|
bool changed = runOnBlock(block, cfg, ac, vals,
|
|
|
|
classification, wasAnalyzed);
|
2011-07-07 00:21:37 +08:00
|
|
|
++stats.NumBlockVisits;
|
2011-01-15 10:58:47 +08:00
|
|
|
if (changed || !previouslyVisited[block->getBlockID()])
|
|
|
|
worklist.enqueueSuccessors(block);
|
|
|
|
previouslyVisited[block->getBlockID()] = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Run through the blocks one more time, and report uninitialized variabes.
|
|
|
|
for (CFG::const_iterator BI = cfg.begin(), BE = cfg.end(); BI != BE; ++BI) {
|
2011-09-03 03:39:26 +08:00
|
|
|
const CFGBlock *block = *BI;
|
|
|
|
if (wasAnalyzed[block->getBlockID()]) {
|
2012-07-17 08:06:14 +08:00
|
|
|
runOnBlock(block, cfg, ac, vals, classification, wasAnalyzed, &handler);
|
2011-07-07 00:21:37 +08:00
|
|
|
++stats.NumBlockVisits;
|
|
|
|
}
|
2011-01-15 10:58:47 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
UninitVariablesHandler::~UninitVariablesHandler() {}
|