blob: e2d1e8c33156a639bee119ddf9925a869e186069 [file] [log] [blame]
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +00001//=- AnalysisBasedWarnings.cpp - Sema warnings based on libAnalysis -*- 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 defines analysis_warnings::[Policy,Executor].
11// Together they are used by Sema to issue warnings based on inexpensive
12// static analysis algorithms in libAnalysis.
13//
14//===----------------------------------------------------------------------===//
15
Douglas Gregore737f502010-08-12 20:07:10 +000016#include "clang/Sema/AnalysisBasedWarnings.h"
John McCall2d887082010-08-25 22:03:47 +000017#include "clang/Sema/SemaInternal.h"
Ted Kremenek351ba912011-02-23 01:52:04 +000018#include "clang/Sema/ScopeInfo.h"
Ted Kremenekd068aab2010-03-20 21:11:09 +000019#include "clang/Basic/SourceManager.h"
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +000020#include "clang/Basic/SourceLocation.h"
Ted Kremenekfbb178a2011-01-21 19:41:46 +000021#include "clang/Lex/Preprocessor.h"
John McCall7cd088e2010-08-24 07:21:54 +000022#include "clang/AST/DeclObjC.h"
John McCall384aff82010-08-25 07:42:41 +000023#include "clang/AST/DeclCXX.h"
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000024#include "clang/AST/ExprObjC.h"
25#include "clang/AST/ExprCXX.h"
26#include "clang/AST/StmtObjC.h"
27#include "clang/AST/StmtCXX.h"
Ted Kremenek6f417152011-04-04 20:56:00 +000028#include "clang/AST/EvaluatedExprVisitor.h"
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +000029#include "clang/AST/StmtVisitor.h"
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000030#include "clang/Analysis/AnalysisContext.h"
31#include "clang/Analysis/CFG.h"
32#include "clang/Analysis/Analyses/ReachableCode.h"
Ted Kremenek351ba912011-02-23 01:52:04 +000033#include "clang/Analysis/Analyses/CFGReachabilityAnalysis.h"
Caitlin Sadowski402aa062011-09-09 16:11:56 +000034#include "clang/Analysis/Analyses/ThreadSafety.h"
Ted Kremenek351ba912011-02-23 01:52:04 +000035#include "clang/Analysis/CFGStmtMap.h"
Ted Kremenek6f342132011-03-15 03:17:07 +000036#include "clang/Analysis/Analyses/UninitializedValues.h"
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000037#include "llvm/ADT/BitVector.h"
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +000038#include "llvm/ADT/FoldingSet.h"
39#include "llvm/ADT/ImmutableMap.h"
40#include "llvm/ADT/PostOrderIterator.h"
41#include "llvm/ADT/SmallVector.h"
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +000042#include "llvm/ADT/StringRef.h"
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000043#include "llvm/Support/Casting.h"
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +000044#include <algorithm>
45#include <vector>
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000046
47using namespace clang;
48
49//===----------------------------------------------------------------------===//
50// Unreachable code analysis.
51//===----------------------------------------------------------------------===//
52
53namespace {
54 class UnreachableCodeHandler : public reachable_code::Callback {
55 Sema &S;
56 public:
57 UnreachableCodeHandler(Sema &s) : S(s) {}
58
59 void HandleUnreachable(SourceLocation L, SourceRange R1, SourceRange R2) {
60 S.Diag(L, diag::warn_unreachable) << R1 << R2;
61 }
62 };
63}
64
65/// CheckUnreachable - Check for unreachable code.
Ted Kremenek1d26f482011-10-24 01:32:45 +000066static void CheckUnreachable(Sema &S, AnalysisDeclContext &AC) {
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000067 UnreachableCodeHandler UC(S);
68 reachable_code::FindUnreachableCode(AC, UC);
69}
70
71//===----------------------------------------------------------------------===//
72// Check for missing return value.
73//===----------------------------------------------------------------------===//
74
John McCall16565aa2010-05-16 09:34:11 +000075enum ControlFlowKind {
76 UnknownFallThrough,
77 NeverFallThrough,
78 MaybeFallThrough,
79 AlwaysFallThrough,
80 NeverFallThroughOrReturn
81};
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000082
83/// CheckFallThrough - Check that we don't fall off the end of a
84/// Statement that should return a value.
85///
86/// \returns AlwaysFallThrough iff we always fall off the end of the statement,
87/// MaybeFallThrough iff we might or might not fall off the end,
88/// NeverFallThroughOrReturn iff we never fall off the end of the statement or
89/// return. We assume NeverFallThrough iff we never fall off the end of the
90/// statement but we may return. We assume that functions not marked noreturn
91/// will return.
Ted Kremenek1d26f482011-10-24 01:32:45 +000092static ControlFlowKind CheckFallThrough(AnalysisDeclContext &AC) {
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000093 CFG *cfg = AC.getCFG();
John McCall16565aa2010-05-16 09:34:11 +000094 if (cfg == 0) return UnknownFallThrough;
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000095
96 // The CFG leaves in dead things, and we don't want the dead code paths to
97 // confuse us, so we mark all live things first.
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +000098 llvm::BitVector live(cfg->getNumBlockIDs());
Ted Kremenek0f3b4ca2011-08-23 23:05:11 +000099 unsigned count = reachable_code::ScanReachableFromBlock(&cfg->getEntry(),
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000100 live);
101
102 bool AddEHEdges = AC.getAddEHEdges();
103 if (!AddEHEdges && count != cfg->getNumBlockIDs())
104 // When there are things remaining dead, and we didn't add EH edges
105 // from CallExprs to the catch clauses, we have to go back and
106 // mark them as live.
107 for (CFG::iterator I = cfg->begin(), E = cfg->end(); I != E; ++I) {
108 CFGBlock &b = **I;
109 if (!live[b.getBlockID()]) {
110 if (b.pred_begin() == b.pred_end()) {
111 if (b.getTerminator() && isa<CXXTryStmt>(b.getTerminator()))
112 // When not adding EH edges from calls, catch clauses
113 // can otherwise seem dead. Avoid noting them as dead.
Ted Kremenek0f3b4ca2011-08-23 23:05:11 +0000114 count += reachable_code::ScanReachableFromBlock(&b, live);
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000115 continue;
116 }
117 }
118 }
119
120 // Now we know what is live, we check the live precessors of the exit block
121 // and look for fall through paths, being careful to ignore normal returns,
122 // and exceptional paths.
123 bool HasLiveReturn = false;
124 bool HasFakeEdge = false;
125 bool HasPlainEdge = false;
126 bool HasAbnormalEdge = false;
Ted Kremenek90b828a2010-09-09 00:06:07 +0000127
128 // Ignore default cases that aren't likely to be reachable because all
129 // enums in a switch(X) have explicit case statements.
130 CFGBlock::FilterOptions FO;
131 FO.IgnoreDefaultsWithCoveredEnums = 1;
132
133 for (CFGBlock::filtered_pred_iterator
134 I = cfg->getExit().filtered_pred_start_end(FO); I.hasMore(); ++I) {
135 const CFGBlock& B = **I;
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000136 if (!live[B.getBlockID()])
137 continue;
Ted Kremenek5811f592011-01-26 04:49:52 +0000138
Chandler Carruthe05ee6d2011-09-13 09:53:58 +0000139 // Skip blocks which contain an element marked as no-return. They don't
140 // represent actually viable edges into the exit block, so mark them as
141 // abnormal.
142 if (B.hasNoReturnElement()) {
143 HasAbnormalEdge = true;
144 continue;
145 }
146
Ted Kremenek5811f592011-01-26 04:49:52 +0000147 // Destructors can appear after the 'return' in the CFG. This is
148 // normal. We need to look pass the destructors for the return
149 // statement (if it exists).
150 CFGBlock::const_reverse_iterator ri = B.rbegin(), re = B.rend();
Ted Kremenekc9f8f5a2011-03-02 20:32:29 +0000151
Chandler Carruthe05ee6d2011-09-13 09:53:58 +0000152 for ( ; ri != re ; ++ri)
153 if (isa<CFGStmt>(*ri))
Ted Kremenek5811f592011-01-26 04:49:52 +0000154 break;
Chandler Carruthe05ee6d2011-09-13 09:53:58 +0000155
Ted Kremenek5811f592011-01-26 04:49:52 +0000156 // No more CFGElements in the block?
157 if (ri == re) {
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000158 if (B.getTerminator() && isa<CXXTryStmt>(B.getTerminator())) {
159 HasAbnormalEdge = true;
160 continue;
161 }
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000162 // A labeled empty statement, or the entry block...
163 HasPlainEdge = true;
164 continue;
165 }
Ted Kremenekf39e6a32011-01-25 22:50:47 +0000166
Ted Kremenek5811f592011-01-26 04:49:52 +0000167 CFGStmt CS = cast<CFGStmt>(*ri);
Ted Kremenekf1d10d92011-08-23 23:05:04 +0000168 const Stmt *S = CS.getStmt();
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000169 if (isa<ReturnStmt>(S)) {
170 HasLiveReturn = true;
171 continue;
172 }
173 if (isa<ObjCAtThrowStmt>(S)) {
174 HasFakeEdge = true;
175 continue;
176 }
177 if (isa<CXXThrowExpr>(S)) {
178 HasFakeEdge = true;
179 continue;
180 }
181 if (const AsmStmt *AS = dyn_cast<AsmStmt>(S)) {
182 if (AS->isMSAsm()) {
183 HasFakeEdge = true;
184 HasLiveReturn = true;
185 continue;
186 }
187 }
188 if (isa<CXXTryStmt>(S)) {
189 HasAbnormalEdge = true;
190 continue;
191 }
Chandler Carruthe05ee6d2011-09-13 09:53:58 +0000192 if (std::find(B.succ_begin(), B.succ_end(), &cfg->getExit())
193 == B.succ_end()) {
194 HasAbnormalEdge = true;
195 continue;
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000196 }
Chandler Carruthe05ee6d2011-09-13 09:53:58 +0000197
198 HasPlainEdge = true;
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000199 }
200 if (!HasPlainEdge) {
201 if (HasLiveReturn)
202 return NeverFallThrough;
203 return NeverFallThroughOrReturn;
204 }
205 if (HasAbnormalEdge || HasFakeEdge || HasLiveReturn)
206 return MaybeFallThrough;
207 // This says AlwaysFallThrough for calls to functions that are not marked
208 // noreturn, that don't return. If people would like this warning to be more
209 // accurate, such functions should be marked as noreturn.
210 return AlwaysFallThrough;
211}
212
Dan Gohman3c46e8d2010-07-26 21:25:24 +0000213namespace {
214
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000215struct CheckFallThroughDiagnostics {
216 unsigned diag_MaybeFallThrough_HasNoReturn;
217 unsigned diag_MaybeFallThrough_ReturnsNonVoid;
218 unsigned diag_AlwaysFallThrough_HasNoReturn;
219 unsigned diag_AlwaysFallThrough_ReturnsNonVoid;
220 unsigned diag_NeverFallThroughOrReturn;
221 bool funMode;
Argyrios Kyrtzidis08274082010-12-15 18:44:22 +0000222 SourceLocation FuncLoc;
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000223
Douglas Gregorca7eaee2010-04-16 23:28:44 +0000224 static CheckFallThroughDiagnostics MakeForFunction(const Decl *Func) {
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000225 CheckFallThroughDiagnostics D;
Argyrios Kyrtzidis08274082010-12-15 18:44:22 +0000226 D.FuncLoc = Func->getLocation();
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000227 D.diag_MaybeFallThrough_HasNoReturn =
228 diag::warn_falloff_noreturn_function;
229 D.diag_MaybeFallThrough_ReturnsNonVoid =
230 diag::warn_maybe_falloff_nonvoid_function;
231 D.diag_AlwaysFallThrough_HasNoReturn =
232 diag::warn_falloff_noreturn_function;
233 D.diag_AlwaysFallThrough_ReturnsNonVoid =
234 diag::warn_falloff_nonvoid_function;
Douglas Gregorca7eaee2010-04-16 23:28:44 +0000235
236 // Don't suggest that virtual functions be marked "noreturn", since they
237 // might be overridden by non-noreturn functions.
238 bool isVirtualMethod = false;
239 if (const CXXMethodDecl *Method = dyn_cast<CXXMethodDecl>(Func))
240 isVirtualMethod = Method->isVirtual();
241
Douglas Gregorfcdd2cb2011-10-10 18:15:57 +0000242 // Don't suggest that template instantiations be marked "noreturn"
243 bool isTemplateInstantiation = false;
Ted Kremenek75df4ee2011-12-01 00:59:17 +0000244 if (const FunctionDecl *Function = dyn_cast<FunctionDecl>(Func))
245 isTemplateInstantiation = Function->isTemplateInstantiation();
Douglas Gregorfcdd2cb2011-10-10 18:15:57 +0000246
247 if (!isVirtualMethod && !isTemplateInstantiation)
Douglas Gregorca7eaee2010-04-16 23:28:44 +0000248 D.diag_NeverFallThroughOrReturn =
249 diag::warn_suggest_noreturn_function;
250 else
251 D.diag_NeverFallThroughOrReturn = 0;
252
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000253 D.funMode = true;
254 return D;
255 }
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000256
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000257 static CheckFallThroughDiagnostics MakeForBlock() {
258 CheckFallThroughDiagnostics D;
259 D.diag_MaybeFallThrough_HasNoReturn =
260 diag::err_noreturn_block_has_return_expr;
261 D.diag_MaybeFallThrough_ReturnsNonVoid =
262 diag::err_maybe_falloff_nonvoid_block;
263 D.diag_AlwaysFallThrough_HasNoReturn =
264 diag::err_noreturn_block_has_return_expr;
265 D.diag_AlwaysFallThrough_ReturnsNonVoid =
266 diag::err_falloff_nonvoid_block;
267 D.diag_NeverFallThroughOrReturn =
268 diag::warn_suggest_noreturn_block;
269 D.funMode = false;
270 return D;
271 }
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000272
David Blaikied6471f72011-09-25 23:23:43 +0000273 bool checkDiagnostics(DiagnosticsEngine &D, bool ReturnsVoid,
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000274 bool HasNoReturn) const {
275 if (funMode) {
Argyrios Kyrtzidis08274082010-12-15 18:44:22 +0000276 return (ReturnsVoid ||
277 D.getDiagnosticLevel(diag::warn_maybe_falloff_nonvoid_function,
David Blaikied6471f72011-09-25 23:23:43 +0000278 FuncLoc) == DiagnosticsEngine::Ignored)
Argyrios Kyrtzidis08274082010-12-15 18:44:22 +0000279 && (!HasNoReturn ||
280 D.getDiagnosticLevel(diag::warn_noreturn_function_has_return_expr,
David Blaikied6471f72011-09-25 23:23:43 +0000281 FuncLoc) == DiagnosticsEngine::Ignored)
Argyrios Kyrtzidis08274082010-12-15 18:44:22 +0000282 && (!ReturnsVoid ||
283 D.getDiagnosticLevel(diag::warn_suggest_noreturn_block, FuncLoc)
David Blaikied6471f72011-09-25 23:23:43 +0000284 == DiagnosticsEngine::Ignored);
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000285 }
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000286
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000287 // For blocks.
288 return ReturnsVoid && !HasNoReturn
Argyrios Kyrtzidis08274082010-12-15 18:44:22 +0000289 && (!ReturnsVoid ||
290 D.getDiagnosticLevel(diag::warn_suggest_noreturn_block, FuncLoc)
David Blaikied6471f72011-09-25 23:23:43 +0000291 == DiagnosticsEngine::Ignored);
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000292 }
293};
294
Dan Gohman3c46e8d2010-07-26 21:25:24 +0000295}
296
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000297/// CheckFallThroughForFunctionDef - Check that we don't fall off the end of a
298/// function that should return a value. Check that we don't fall off the end
299/// of a noreturn function. We assume that functions and blocks not marked
300/// noreturn will return.
301static void CheckFallThroughForBody(Sema &S, const Decl *D, const Stmt *Body,
Ted Kremenek3ed6fc02011-02-23 01:51:48 +0000302 const BlockExpr *blkExpr,
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000303 const CheckFallThroughDiagnostics& CD,
Ted Kremenek1d26f482011-10-24 01:32:45 +0000304 AnalysisDeclContext &AC) {
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000305
306 bool ReturnsVoid = false;
307 bool HasNoReturn = false;
308
309 if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(D)) {
310 ReturnsVoid = FD->getResultType()->isVoidType();
311 HasNoReturn = FD->hasAttr<NoReturnAttr>() ||
Rafael Espindola264ba482010-03-30 20:24:48 +0000312 FD->getType()->getAs<FunctionType>()->getNoReturnAttr();
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000313 }
314 else if (const ObjCMethodDecl *MD = dyn_cast<ObjCMethodDecl>(D)) {
315 ReturnsVoid = MD->getResultType()->isVoidType();
316 HasNoReturn = MD->hasAttr<NoReturnAttr>();
317 }
318 else if (isa<BlockDecl>(D)) {
Ted Kremenek3ed6fc02011-02-23 01:51:48 +0000319 QualType BlockTy = blkExpr->getType();
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000320 if (const FunctionType *FT =
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000321 BlockTy->getPointeeType()->getAs<FunctionType>()) {
322 if (FT->getResultType()->isVoidType())
323 ReturnsVoid = true;
324 if (FT->getNoReturnAttr())
325 HasNoReturn = true;
326 }
327 }
328
David Blaikied6471f72011-09-25 23:23:43 +0000329 DiagnosticsEngine &Diags = S.getDiagnostics();
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000330
331 // Short circuit for compilation speed.
332 if (CD.checkDiagnostics(Diags, ReturnsVoid, HasNoReturn))
333 return;
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000334
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000335 // FIXME: Function try block
336 if (const CompoundStmt *Compound = dyn_cast<CompoundStmt>(Body)) {
337 switch (CheckFallThrough(AC)) {
John McCall16565aa2010-05-16 09:34:11 +0000338 case UnknownFallThrough:
339 break;
340
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000341 case MaybeFallThrough:
342 if (HasNoReturn)
343 S.Diag(Compound->getRBracLoc(),
344 CD.diag_MaybeFallThrough_HasNoReturn);
345 else if (!ReturnsVoid)
346 S.Diag(Compound->getRBracLoc(),
347 CD.diag_MaybeFallThrough_ReturnsNonVoid);
348 break;
349 case AlwaysFallThrough:
350 if (HasNoReturn)
351 S.Diag(Compound->getRBracLoc(),
352 CD.diag_AlwaysFallThrough_HasNoReturn);
353 else if (!ReturnsVoid)
354 S.Diag(Compound->getRBracLoc(),
355 CD.diag_AlwaysFallThrough_ReturnsNonVoid);
356 break;
357 case NeverFallThroughOrReturn:
Chandler Carruthb0656ec2011-08-31 09:01:53 +0000358 if (ReturnsVoid && !HasNoReturn && CD.diag_NeverFallThroughOrReturn) {
359 if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(D)) {
360 S.Diag(Compound->getLBracLoc(), CD.diag_NeverFallThroughOrReturn)
Douglas Gregorb3321092011-09-10 00:56:20 +0000361 << 0 << FD;
362 } else if (const ObjCMethodDecl *MD = dyn_cast<ObjCMethodDecl>(D)) {
363 S.Diag(Compound->getLBracLoc(), CD.diag_NeverFallThroughOrReturn)
364 << 1 << MD;
Chandler Carruthb0656ec2011-08-31 09:01:53 +0000365 } else {
366 S.Diag(Compound->getLBracLoc(), CD.diag_NeverFallThroughOrReturn);
367 }
368 }
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000369 break;
370 case NeverFallThrough:
371 break;
372 }
373 }
374}
375
376//===----------------------------------------------------------------------===//
Ted Kremenek610068c2011-01-15 02:58:47 +0000377// -Wuninitialized
378//===----------------------------------------------------------------------===//
379
Ted Kremenek6f417152011-04-04 20:56:00 +0000380namespace {
Chandler Carruth9f649462011-04-05 06:48:00 +0000381/// ContainsReference - A visitor class to search for references to
382/// a particular declaration (the needle) within any evaluated component of an
383/// expression (recursively).
Ted Kremenek6f417152011-04-04 20:56:00 +0000384class ContainsReference : public EvaluatedExprVisitor<ContainsReference> {
Chandler Carruth9f649462011-04-05 06:48:00 +0000385 bool FoundReference;
386 const DeclRefExpr *Needle;
387
Ted Kremenek6f417152011-04-04 20:56:00 +0000388public:
Chandler Carruth9f649462011-04-05 06:48:00 +0000389 ContainsReference(ASTContext &Context, const DeclRefExpr *Needle)
390 : EvaluatedExprVisitor<ContainsReference>(Context),
391 FoundReference(false), Needle(Needle) {}
392
393 void VisitExpr(Expr *E) {
Ted Kremenek6f417152011-04-04 20:56:00 +0000394 // Stop evaluating if we already have a reference.
Chandler Carruth9f649462011-04-05 06:48:00 +0000395 if (FoundReference)
Ted Kremenek6f417152011-04-04 20:56:00 +0000396 return;
Chandler Carruth9f649462011-04-05 06:48:00 +0000397
398 EvaluatedExprVisitor<ContainsReference>::VisitExpr(E);
Ted Kremenek6f417152011-04-04 20:56:00 +0000399 }
Chandler Carruth9f649462011-04-05 06:48:00 +0000400
401 void VisitDeclRefExpr(DeclRefExpr *E) {
402 if (E == Needle)
403 FoundReference = true;
404 else
405 EvaluatedExprVisitor<ContainsReference>::VisitDeclRefExpr(E);
Ted Kremenek6f417152011-04-04 20:56:00 +0000406 }
Chandler Carruth9f649462011-04-05 06:48:00 +0000407
408 bool doesContainReference() const { return FoundReference; }
Ted Kremenek6f417152011-04-04 20:56:00 +0000409};
410}
411
David Blaikie4f4f3492011-09-10 05:35:08 +0000412static bool SuggestInitializationFixit(Sema &S, const VarDecl *VD) {
413 // Don't issue a fixit if there is already an initializer.
414 if (VD->getInit())
415 return false;
416
417 // Suggest possible initialization (if any).
David Blaikie4f4f3492011-09-10 05:35:08 +0000418 QualType VariableTy = VD->getType().getCanonicalType();
Richard Smith7984de32012-01-12 23:53:29 +0000419 const char *Init = S.getFixItZeroInitializerForType(VariableTy);
420 if (!Init)
David Blaikie4f4f3492011-09-10 05:35:08 +0000421 return false;
David Blaikie4f4f3492011-09-10 05:35:08 +0000422
Richard Smith7984de32012-01-12 23:53:29 +0000423 SourceLocation Loc = S.PP.getLocForEndOfToken(VD->getLocEnd());
424 S.Diag(Loc, diag::note_var_fixit_add_initialization) << VD->getDeclName()
425 << FixItHint::CreateInsertion(Loc, Init);
426 return true;
David Blaikie4f4f3492011-09-10 05:35:08 +0000427}
428
Chandler Carruth262d50e2011-04-05 18:27:05 +0000429/// DiagnoseUninitializedUse -- Helper function for diagnosing uses of an
430/// uninitialized variable. This manages the different forms of diagnostic
431/// emitted for particular types of uses. Returns true if the use was diagnosed
432/// as a warning. If a pariticular use is one we omit warnings for, returns
433/// false.
434static bool DiagnoseUninitializedUse(Sema &S, const VarDecl *VD,
Ted Kremenek9e761722011-10-13 18:50:06 +0000435 const Expr *E, bool isAlwaysUninit,
436 bool alwaysReportSelfInit = false) {
Chandler Carruth4c4983b2011-04-05 18:18:05 +0000437 bool isSelfInit = false;
438
439 if (const DeclRefExpr *DRE = dyn_cast<DeclRefExpr>(E)) {
440 if (isAlwaysUninit) {
441 // Inspect the initializer of the variable declaration which is
442 // being referenced prior to its initialization. We emit
443 // specialized diagnostics for self-initialization, and we
444 // specifically avoid warning about self references which take the
445 // form of:
446 //
447 // int x = x;
448 //
449 // This is used to indicate to GCC that 'x' is intentionally left
450 // uninitialized. Proven code paths which access 'x' in
451 // an uninitialized state after this will still warn.
452 //
453 // TODO: Should we suppress maybe-uninitialized warnings for
454 // variables initialized in this way?
455 if (const Expr *Initializer = VD->getInit()) {
Ted Kremenek9e761722011-10-13 18:50:06 +0000456 if (!alwaysReportSelfInit && DRE == Initializer->IgnoreParenImpCasts())
Chandler Carruth262d50e2011-04-05 18:27:05 +0000457 return false;
Chandler Carruth4c4983b2011-04-05 18:18:05 +0000458
459 ContainsReference CR(S.Context, DRE);
460 CR.Visit(const_cast<Expr*>(Initializer));
461 isSelfInit = CR.doesContainReference();
462 }
463 if (isSelfInit) {
464 S.Diag(DRE->getLocStart(),
465 diag::warn_uninit_self_reference_in_init)
466 << VD->getDeclName() << VD->getLocation() << DRE->getSourceRange();
467 } else {
468 S.Diag(DRE->getLocStart(), diag::warn_uninit_var)
469 << VD->getDeclName() << DRE->getSourceRange();
470 }
471 } else {
472 S.Diag(DRE->getLocStart(), diag::warn_maybe_uninit_var)
473 << VD->getDeclName() << DRE->getSourceRange();
474 }
475 } else {
476 const BlockExpr *BE = cast<BlockExpr>(E);
477 S.Diag(BE->getLocStart(),
478 isAlwaysUninit ? diag::warn_uninit_var_captured_by_block
479 : diag::warn_maybe_uninit_var_captured_by_block)
480 << VD->getDeclName();
481 }
482
483 // Report where the variable was declared when the use wasn't within
David Blaikie4f4f3492011-09-10 05:35:08 +0000484 // the initializer of that declaration & we didn't already suggest
485 // an initialization fixit.
486 if (!isSelfInit && !SuggestInitializationFixit(S, VD))
Chandler Carruth4c4983b2011-04-05 18:18:05 +0000487 S.Diag(VD->getLocStart(), diag::note_uninit_var_def)
488 << VD->getDeclName();
489
Chandler Carruth262d50e2011-04-05 18:27:05 +0000490 return true;
Chandler Carruth64fb9592011-04-05 18:18:08 +0000491}
492
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000493typedef std::pair<const Expr*, bool> UninitUse;
494
Ted Kremenek610068c2011-01-15 02:58:47 +0000495namespace {
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000496struct SLocSort {
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000497 bool operator()(const UninitUse &a, const UninitUse &b) {
498 SourceLocation aLoc = a.first->getLocStart();
499 SourceLocation bLoc = b.first->getLocStart();
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000500 return aLoc.getRawEncoding() < bLoc.getRawEncoding();
501 }
502};
503
Ted Kremenek610068c2011-01-15 02:58:47 +0000504class UninitValsDiagReporter : public UninitVariablesHandler {
505 Sema &S;
Chris Lattner5f9e2722011-07-23 10:55:15 +0000506 typedef SmallVector<UninitUse, 2> UsesVec;
Ted Kremenek9e761722011-10-13 18:50:06 +0000507 typedef llvm::DenseMap<const VarDecl *, std::pair<UsesVec*, bool> > UsesMap;
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000508 UsesMap *uses;
509
Ted Kremenek610068c2011-01-15 02:58:47 +0000510public:
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000511 UninitValsDiagReporter(Sema &S) : S(S), uses(0) {}
512 ~UninitValsDiagReporter() {
513 flushDiagnostics();
514 }
Ted Kremenek9e761722011-10-13 18:50:06 +0000515
516 std::pair<UsesVec*, bool> &getUses(const VarDecl *vd) {
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000517 if (!uses)
518 uses = new UsesMap();
Ted Kremenek9e761722011-10-13 18:50:06 +0000519
520 UsesMap::mapped_type &V = (*uses)[vd];
521 UsesVec *&vec = V.first;
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000522 if (!vec)
523 vec = new UsesVec();
524
Ted Kremenek9e761722011-10-13 18:50:06 +0000525 return V;
526 }
527
528 void handleUseOfUninitVariable(const Expr *ex, const VarDecl *vd,
529 bool isAlwaysUninit) {
530 getUses(vd).first->push_back(std::make_pair(ex, isAlwaysUninit));
531 }
532
533 void handleSelfInit(const VarDecl *vd) {
534 getUses(vd).second = true;
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000535 }
536
537 void flushDiagnostics() {
538 if (!uses)
539 return;
Ted Kremenek609e3172011-02-02 23:35:53 +0000540
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000541 for (UsesMap::iterator i = uses->begin(), e = uses->end(); i != e; ++i) {
542 const VarDecl *vd = i->first;
Ted Kremenek9e761722011-10-13 18:50:06 +0000543 const UsesMap::mapped_type &V = i->second;
Ted Kremenek609e3172011-02-02 23:35:53 +0000544
Ted Kremenek9e761722011-10-13 18:50:06 +0000545 UsesVec *vec = V.first;
546 bool hasSelfInit = V.second;
547
548 // Specially handle the case where we have uses of an uninitialized
549 // variable, but the root cause is an idiomatic self-init. We want
550 // to report the diagnostic at the self-init since that is the root cause.
Matt Beaumont-Gay0d381812011-10-19 18:53:03 +0000551 if (!vec->empty() && hasSelfInit && hasAlwaysUninitializedUse(vec))
Ted Kremenek9e761722011-10-13 18:50:06 +0000552 DiagnoseUninitializedUse(S, vd, vd->getInit()->IgnoreParenCasts(),
Matt Beaumont-Gay0d381812011-10-19 18:53:03 +0000553 /* isAlwaysUninit */ true,
554 /* alwaysReportSelfInit */ true);
Ted Kremenek9e761722011-10-13 18:50:06 +0000555 else {
556 // Sort the uses by their SourceLocations. While not strictly
557 // guaranteed to produce them in line/column order, this will provide
558 // a stable ordering.
559 std::sort(vec->begin(), vec->end(), SLocSort());
560
561 for (UsesVec::iterator vi = vec->begin(), ve = vec->end(); vi != ve;
562 ++vi) {
563 if (DiagnoseUninitializedUse(S, vd, vi->first,
564 /*isAlwaysUninit=*/vi->second))
565 // Skip further diagnostics for this variable. We try to warn only
566 // on the first point at which a variable is used uninitialized.
567 break;
568 }
Chandler Carruth64fb9592011-04-05 18:18:08 +0000569 }
Ted Kremenek9e761722011-10-13 18:50:06 +0000570
571 // Release the uses vector.
Ted Kremenek94b1b4d2011-01-21 19:41:41 +0000572 delete vec;
573 }
574 delete uses;
Ted Kremenek610068c2011-01-15 02:58:47 +0000575 }
Matt Beaumont-Gay0d381812011-10-19 18:53:03 +0000576
577private:
578 static bool hasAlwaysUninitializedUse(const UsesVec* vec) {
579 for (UsesVec::const_iterator i = vec->begin(), e = vec->end(); i != e; ++i) {
580 if (i->second) {
581 return true;
582 }
583 }
584 return false;
585}
Ted Kremenek610068c2011-01-15 02:58:47 +0000586};
587}
588
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +0000589
590//===----------------------------------------------------------------------===//
591// -Wthread-safety
592//===----------------------------------------------------------------------===//
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000593namespace clang {
594namespace thread_safety {
595typedef std::pair<SourceLocation, PartialDiagnostic> DelayedDiag;
596typedef llvm::SmallVector<DelayedDiag, 4> DiagList;
597
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000598struct SortDiagBySourceLocation {
599 Sema &S;
600 SortDiagBySourceLocation(Sema &S) : S(S) {}
601
602 bool operator()(const DelayedDiag &left, const DelayedDiag &right) {
603 // Although this call will be slow, this is only called when outputting
604 // multiple warnings.
605 return S.getSourceManager().isBeforeInTranslationUnit(left.first,
606 right.first);
607 }
608};
609
David Blaikie99ba9e32011-12-20 02:48:34 +0000610namespace {
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000611class ThreadSafetyReporter : public clang::thread_safety::ThreadSafetyHandler {
612 Sema &S;
613 DiagList Warnings;
DeLesley Hutchinsf1ac6372011-10-21 18:10:14 +0000614 SourceLocation FunLocation;
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000615
616 // Helper functions
617 void warnLockMismatch(unsigned DiagID, Name LockName, SourceLocation Loc) {
DeLesley Hutchinsf1ac6372011-10-21 18:10:14 +0000618 // Gracefully handle rare cases when the analysis can't get a more
619 // precise source location.
620 if (!Loc.isValid())
621 Loc = FunLocation;
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000622 PartialDiagnostic Warning = S.PDiag(DiagID) << LockName;
623 Warnings.push_back(DelayedDiag(Loc, Warning));
624 }
625
626 public:
DeLesley Hutchinsf1ac6372011-10-21 18:10:14 +0000627 ThreadSafetyReporter(Sema &S, SourceLocation FL)
628 : S(S), FunLocation(FL) {}
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000629
630 /// \brief Emit all buffered diagnostics in order of sourcelocation.
631 /// We need to output diagnostics produced while iterating through
632 /// the lockset in deterministic order, so this function orders diagnostics
633 /// and outputs them.
634 void emitDiagnostics() {
635 SortDiagBySourceLocation SortDiagBySL(S);
636 sort(Warnings.begin(), Warnings.end(), SortDiagBySL);
637 for (DiagList::iterator I = Warnings.begin(), E = Warnings.end();
638 I != E; ++I)
639 S.Diag(I->first, I->second);
640 }
641
Caitlin Sadowski99107eb2011-09-09 16:21:55 +0000642 void handleInvalidLockExp(SourceLocation Loc) {
643 PartialDiagnostic Warning = S.PDiag(diag::warn_cannot_resolve_lock) << Loc;
644 Warnings.push_back(DelayedDiag(Loc, Warning));
645 }
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000646 void handleUnmatchedUnlock(Name LockName, SourceLocation Loc) {
647 warnLockMismatch(diag::warn_unlock_but_no_lock, LockName, Loc);
648 }
649
650 void handleDoubleLock(Name LockName, SourceLocation Loc) {
651 warnLockMismatch(diag::warn_double_lock, LockName, Loc);
652 }
653
Caitlin Sadowski4e4bc752011-09-15 17:25:19 +0000654 void handleMutexHeldEndOfScope(Name LockName, SourceLocation Loc,
655 LockErrorKind LEK){
656 unsigned DiagID = 0;
657 switch (LEK) {
658 case LEK_LockedSomePredecessors:
659 DiagID = diag::warn_lock_at_end_of_scope;
660 break;
661 case LEK_LockedSomeLoopIterations:
662 DiagID = diag::warn_expecting_lock_held_on_loop;
663 break;
664 case LEK_LockedAtEndOfFunction:
665 DiagID = diag::warn_no_unlock;
666 break;
667 }
668 warnLockMismatch(DiagID, LockName, Loc);
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000669 }
670
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000671
672 void handleExclusiveAndShared(Name LockName, SourceLocation Loc1,
673 SourceLocation Loc2) {
674 PartialDiagnostic Warning =
675 S.PDiag(diag::warn_lock_exclusive_and_shared) << LockName;
676 PartialDiagnostic Note =
677 S.PDiag(diag::note_lock_exclusive_and_shared) << LockName;
678 Warnings.push_back(DelayedDiag(Loc1, Warning));
679 Warnings.push_back(DelayedDiag(Loc2, Note));
680 }
681
682 void handleNoMutexHeld(const NamedDecl *D, ProtectedOperationKind POK,
683 AccessKind AK, SourceLocation Loc) {
Caitlin Sadowskidf8327c2011-09-14 20:09:09 +0000684 assert((POK == POK_VarAccess || POK == POK_VarDereference)
685 && "Only works for variables");
686 unsigned DiagID = POK == POK_VarAccess?
687 diag::warn_variable_requires_any_lock:
688 diag::warn_var_deref_requires_any_lock;
689 PartialDiagnostic Warning = S.PDiag(DiagID)
690 << D->getName() << getLockKindFromAccessKind(AK);
691 Warnings.push_back(DelayedDiag(Loc, Warning));
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000692 }
693
694 void handleMutexNotHeld(const NamedDecl *D, ProtectedOperationKind POK,
695 Name LockName, LockKind LK, SourceLocation Loc) {
Caitlin Sadowskie87158d2011-09-13 18:01:58 +0000696 unsigned DiagID = 0;
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000697 switch (POK) {
698 case POK_VarAccess:
699 DiagID = diag::warn_variable_requires_lock;
700 break;
701 case POK_VarDereference:
702 DiagID = diag::warn_var_deref_requires_lock;
703 break;
704 case POK_FunctionCall:
705 DiagID = diag::warn_fun_requires_lock;
706 break;
707 }
708 PartialDiagnostic Warning = S.PDiag(DiagID)
Caitlin Sadowskidf8327c2011-09-14 20:09:09 +0000709 << D->getName() << LockName << LK;
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000710 Warnings.push_back(DelayedDiag(Loc, Warning));
711 }
712
713 void handleFunExcludesLock(Name FunName, Name LockName, SourceLocation Loc) {
714 PartialDiagnostic Warning =
715 S.PDiag(diag::warn_fun_excludes_mutex) << FunName << LockName;
716 Warnings.push_back(DelayedDiag(Loc, Warning));
717 }
718};
719}
720}
David Blaikie99ba9e32011-12-20 02:48:34 +0000721}
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000722
Ted Kremenek610068c2011-01-15 02:58:47 +0000723//===----------------------------------------------------------------------===//
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000724// AnalysisBasedWarnings - Worker object used by Sema to execute analysis-based
725// warnings on a function, method, or block.
726//===----------------------------------------------------------------------===//
727
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000728clang::sema::AnalysisBasedWarnings::Policy::Policy() {
729 enableCheckFallThrough = 1;
730 enableCheckUnreachable = 0;
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +0000731 enableThreadSafetyAnalysis = 0;
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000732}
733
Chandler Carruth5d989942011-07-06 16:21:37 +0000734clang::sema::AnalysisBasedWarnings::AnalysisBasedWarnings(Sema &s)
735 : S(s),
736 NumFunctionsAnalyzed(0),
Benjamin Kramer54cf3412011-07-08 20:38:53 +0000737 NumFunctionsWithBadCFGs(0),
Chandler Carruth5d989942011-07-06 16:21:37 +0000738 NumCFGBlocks(0),
Benjamin Kramer54cf3412011-07-08 20:38:53 +0000739 MaxCFGBlocksPerFunction(0),
740 NumUninitAnalysisFunctions(0),
741 NumUninitAnalysisVariables(0),
742 MaxUninitAnalysisVariablesPerFunction(0),
743 NumUninitAnalysisBlockVisits(0),
744 MaxUninitAnalysisBlockVisitsPerFunction(0) {
David Blaikied6471f72011-09-25 23:23:43 +0000745 DiagnosticsEngine &D = S.getDiagnostics();
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000746 DefaultPolicy.enableCheckUnreachable = (unsigned)
Argyrios Kyrtzidis08274082010-12-15 18:44:22 +0000747 (D.getDiagnosticLevel(diag::warn_unreachable, SourceLocation()) !=
David Blaikied6471f72011-09-25 23:23:43 +0000748 DiagnosticsEngine::Ignored);
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +0000749 DefaultPolicy.enableThreadSafetyAnalysis = (unsigned)
750 (D.getDiagnosticLevel(diag::warn_double_lock, SourceLocation()) !=
David Blaikied6471f72011-09-25 23:23:43 +0000751 DiagnosticsEngine::Ignored);
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +0000752
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000753}
754
Ted Kremenek351ba912011-02-23 01:52:04 +0000755static void flushDiagnostics(Sema &S, sema::FunctionScopeInfo *fscope) {
Chris Lattner5f9e2722011-07-23 10:55:15 +0000756 for (SmallVectorImpl<sema::PossiblyUnreachableDiag>::iterator
Ted Kremenek351ba912011-02-23 01:52:04 +0000757 i = fscope->PossiblyUnreachableDiags.begin(),
758 e = fscope->PossiblyUnreachableDiags.end();
759 i != e; ++i) {
760 const sema::PossiblyUnreachableDiag &D = *i;
761 S.Diag(D.Loc, D.PD);
762 }
763}
764
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000765void clang::sema::
766AnalysisBasedWarnings::IssueWarnings(sema::AnalysisBasedWarnings::Policy P,
Ted Kremenek283a3582011-02-23 01:51:53 +0000767 sema::FunctionScopeInfo *fscope,
Ted Kremenek3ed6fc02011-02-23 01:51:48 +0000768 const Decl *D, const BlockExpr *blkExpr) {
Ted Kremenekd068aab2010-03-20 21:11:09 +0000769
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000770 // We avoid doing analysis-based warnings when there are errors for
771 // two reasons:
772 // (1) The CFGs often can't be constructed (if the body is invalid), so
773 // don't bother trying.
774 // (2) The code already has problems; running the analysis just takes more
775 // time.
David Blaikied6471f72011-09-25 23:23:43 +0000776 DiagnosticsEngine &Diags = S.getDiagnostics();
Ted Kremenek99e81922010-04-30 21:49:25 +0000777
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000778 // Do not do any analysis for declarations in system headers if we are
779 // going to just ignore them.
Ted Kremenek99e81922010-04-30 21:49:25 +0000780 if (Diags.getSuppressSystemWarnings() &&
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000781 S.SourceMgr.isInSystemHeader(D->getLocation()))
782 return;
783
John McCalle0054f62010-08-25 05:56:39 +0000784 // For code in dependent contexts, we'll do this at instantiation time.
785 if (cast<DeclContext>(D)->isDependentContext())
786 return;
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000787
Ted Kremenek351ba912011-02-23 01:52:04 +0000788 if (Diags.hasErrorOccurred() || Diags.hasFatalErrorOccurred()) {
789 // Flush out any possibly unreachable diagnostics.
790 flushDiagnostics(S, fscope);
791 return;
792 }
793
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000794 const Stmt *Body = D->getBody();
795 assert(Body);
796
Ted Kremenek1d26f482011-10-24 01:32:45 +0000797 AnalysisDeclContext AC(/* AnalysisDeclContextManager */ 0, D, 0);
Ted Kremenekbc5cb8a2011-07-21 05:22:47 +0000798
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000799 // Don't generate EH edges for CallExprs as we'd like to avoid the n^2
800 // explosion for destrutors that can result and the compile time hit.
Ted Kremenekbc5cb8a2011-07-21 05:22:47 +0000801 AC.getCFGBuildOptions().PruneTriviallyFalseEdges = true;
802 AC.getCFGBuildOptions().AddEHEdges = false;
803 AC.getCFGBuildOptions().AddInitializers = true;
804 AC.getCFGBuildOptions().AddImplicitDtors = true;
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000805
806 // Force that certain expressions appear as CFGElements in the CFG. This
807 // is used to speed up various analyses.
808 // FIXME: This isn't the right factoring. This is here for initial
809 // prototyping, but we need a way for analyses to say what expressions they
810 // expect to always be CFGElements and then fill in the BuildOptions
811 // appropriately. This is essentially a layering violation.
DeLesley Hutchins1fa3c062011-12-08 20:23:06 +0000812 if (P.enableCheckUnreachable || P.enableThreadSafetyAnalysis) {
813 // Unreachable code analysis and thread safety require a linearized CFG.
Ted Kremenek0f3b4ca2011-08-23 23:05:11 +0000814 AC.getCFGBuildOptions().setAllAlwaysAdd();
815 }
816 else {
817 AC.getCFGBuildOptions()
818 .setAlwaysAdd(Stmt::BinaryOperatorClass)
819 .setAlwaysAdd(Stmt::BlockExprClass)
820 .setAlwaysAdd(Stmt::CStyleCastExprClass)
821 .setAlwaysAdd(Stmt::DeclRefExprClass)
822 .setAlwaysAdd(Stmt::ImplicitCastExprClass)
823 .setAlwaysAdd(Stmt::UnaryOperatorClass);
824 }
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000825
Ted Kremenekbc5cb8a2011-07-21 05:22:47 +0000826 // Construct the analysis context with the specified CFG build options.
827
Ted Kremenek351ba912011-02-23 01:52:04 +0000828 // Emit delayed diagnostics.
829 if (!fscope->PossiblyUnreachableDiags.empty()) {
830 bool analyzed = false;
Ted Kremenek0d28d362011-03-10 03:50:34 +0000831
832 // Register the expressions with the CFGBuilder.
Chris Lattner5f9e2722011-07-23 10:55:15 +0000833 for (SmallVectorImpl<sema::PossiblyUnreachableDiag>::iterator
Ted Kremenek0d28d362011-03-10 03:50:34 +0000834 i = fscope->PossiblyUnreachableDiags.begin(),
835 e = fscope->PossiblyUnreachableDiags.end();
836 i != e; ++i) {
837 if (const Stmt *stmt = i->stmt)
838 AC.registerForcedBlockExpression(stmt);
839 }
840
841 if (AC.getCFG()) {
842 analyzed = true;
Chris Lattner5f9e2722011-07-23 10:55:15 +0000843 for (SmallVectorImpl<sema::PossiblyUnreachableDiag>::iterator
Ted Kremenek0d28d362011-03-10 03:50:34 +0000844 i = fscope->PossiblyUnreachableDiags.begin(),
845 e = fscope->PossiblyUnreachableDiags.end();
846 i != e; ++i)
847 {
848 const sema::PossiblyUnreachableDiag &D = *i;
849 bool processed = false;
850 if (const Stmt *stmt = i->stmt) {
851 const CFGBlock *block = AC.getBlockForRegisteredExpression(stmt);
Eli Friedman71b8fb52012-01-21 01:01:51 +0000852 CFGReverseBlockReachabilityAnalysis *cra =
853 AC.getCFGReachablityAnalysis();
854 // FIXME: We should be able to assert that block is non-null, but
855 // the CFG analysis can skip potentially-evaluated expressions in
856 // edge cases; see test/Sema/vla-2.c.
857 if (block && cra) {
Ted Kremenek351ba912011-02-23 01:52:04 +0000858 // Can this block be reached from the entrance?
Ted Kremenek0d28d362011-03-10 03:50:34 +0000859 if (cra->isReachable(&AC.getCFG()->getEntry(), block))
Ted Kremenek351ba912011-02-23 01:52:04 +0000860 S.Diag(D.Loc, D.PD);
Ted Kremenek0d28d362011-03-10 03:50:34 +0000861 processed = true;
Ted Kremenek351ba912011-02-23 01:52:04 +0000862 }
863 }
Ted Kremenek0d28d362011-03-10 03:50:34 +0000864 if (!processed) {
865 // Emit the warning anyway if we cannot map to a basic block.
866 S.Diag(D.Loc, D.PD);
867 }
Ted Kremenek351ba912011-02-23 01:52:04 +0000868 }
Ted Kremenek0d28d362011-03-10 03:50:34 +0000869 }
Ted Kremenek351ba912011-02-23 01:52:04 +0000870
871 if (!analyzed)
872 flushDiagnostics(S, fscope);
873 }
874
875
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000876 // Warning: check missing 'return'
Ted Kremenekd064fdc2010-03-23 00:13:23 +0000877 if (P.enableCheckFallThrough) {
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000878 const CheckFallThroughDiagnostics &CD =
879 (isa<BlockDecl>(D) ? CheckFallThroughDiagnostics::MakeForBlock()
Douglas Gregorca7eaee2010-04-16 23:28:44 +0000880 : CheckFallThroughDiagnostics::MakeForFunction(D));
Ted Kremenek3ed6fc02011-02-23 01:51:48 +0000881 CheckFallThroughForBody(S, D, Body, blkExpr, CD, AC);
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000882 }
883
884 // Warning: check for unreachable code
Ted Kremenek5dfee062011-11-30 21:22:09 +0000885 if (P.enableCheckUnreachable) {
886 // Only check for unreachable code on non-template instantiations.
887 // Different template instantiations can effectively change the control-flow
888 // and it is very difficult to prove that a snippet of code in a template
889 // is unreachable for all instantiations.
Ted Kremenek75df4ee2011-12-01 00:59:17 +0000890 bool isTemplateInstantiation = false;
891 if (const FunctionDecl *Function = dyn_cast<FunctionDecl>(D))
892 isTemplateInstantiation = Function->isTemplateInstantiation();
893 if (!isTemplateInstantiation)
Ted Kremenek5dfee062011-11-30 21:22:09 +0000894 CheckUnreachable(S, AC);
895 }
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000896
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +0000897 // Check for thread safety violations
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000898 if (P.enableThreadSafetyAnalysis) {
DeLesley Hutchinsf1ac6372011-10-21 18:10:14 +0000899 SourceLocation FL = AC.getDecl()->getLocation();
900 thread_safety::ThreadSafetyReporter Reporter(S, FL);
Caitlin Sadowski75f23ae2011-09-09 16:04:02 +0000901 thread_safety::runThreadSafetyAnalysis(AC, Reporter);
902 Reporter.emitDiagnostics();
903 }
Caitlin Sadowski3ac1fbc2011-08-23 18:46:34 +0000904
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000905 if (Diags.getDiagnosticLevel(diag::warn_uninit_var, D->getLocStart())
David Blaikied6471f72011-09-25 23:23:43 +0000906 != DiagnosticsEngine::Ignored ||
Ted Kremenek76709bf2011-03-15 05:22:28 +0000907 Diags.getDiagnosticLevel(diag::warn_maybe_uninit_var, D->getLocStart())
David Blaikied6471f72011-09-25 23:23:43 +0000908 != DiagnosticsEngine::Ignored) {
Ted Kremenekc5e43c12011-03-17 05:29:57 +0000909 if (CFG *cfg = AC.getCFG()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000910 UninitValsDiagReporter reporter(S);
Fariborz Jahanian57080fb2011-07-16 18:31:33 +0000911 UninitVariablesAnalysisStats stats;
Benjamin Kramer12efd572011-07-16 20:13:06 +0000912 std::memset(&stats, 0, sizeof(UninitVariablesAnalysisStats));
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000913 runUninitializedVariablesAnalysis(*cast<DeclContext>(D), *cfg, AC,
Chandler Carruth5d989942011-07-06 16:21:37 +0000914 reporter, stats);
915
916 if (S.CollectStats && stats.NumVariablesAnalyzed > 0) {
917 ++NumUninitAnalysisFunctions;
918 NumUninitAnalysisVariables += stats.NumVariablesAnalyzed;
919 NumUninitAnalysisBlockVisits += stats.NumBlockVisits;
920 MaxUninitAnalysisVariablesPerFunction =
921 std::max(MaxUninitAnalysisVariablesPerFunction,
922 stats.NumVariablesAnalyzed);
923 MaxUninitAnalysisBlockVisitsPerFunction =
924 std::max(MaxUninitAnalysisBlockVisitsPerFunction,
925 stats.NumBlockVisits);
926 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000927 }
928 }
Chandler Carruth5d989942011-07-06 16:21:37 +0000929
930 // Collect statistics about the CFG if it was built.
931 if (S.CollectStats && AC.isCFGBuilt()) {
932 ++NumFunctionsAnalyzed;
933 if (CFG *cfg = AC.getCFG()) {
934 // If we successfully built a CFG for this context, record some more
935 // detail information about it.
Chandler Carruth3ea4c492011-07-06 22:21:45 +0000936 NumCFGBlocks += cfg->getNumBlockIDs();
Chandler Carruth5d989942011-07-06 16:21:37 +0000937 MaxCFGBlocksPerFunction = std::max(MaxCFGBlocksPerFunction,
Chandler Carruth3ea4c492011-07-06 22:21:45 +0000938 cfg->getNumBlockIDs());
Chandler Carruth5d989942011-07-06 16:21:37 +0000939 } else {
940 ++NumFunctionsWithBadCFGs;
941 }
942 }
943}
944
945void clang::sema::AnalysisBasedWarnings::PrintStats() const {
946 llvm::errs() << "\n*** Analysis Based Warnings Stats:\n";
947
948 unsigned NumCFGsBuilt = NumFunctionsAnalyzed - NumFunctionsWithBadCFGs;
949 unsigned AvgCFGBlocksPerFunction =
950 !NumCFGsBuilt ? 0 : NumCFGBlocks/NumCFGsBuilt;
951 llvm::errs() << NumFunctionsAnalyzed << " functions analyzed ("
952 << NumFunctionsWithBadCFGs << " w/o CFGs).\n"
953 << " " << NumCFGBlocks << " CFG blocks built.\n"
954 << " " << AvgCFGBlocksPerFunction
955 << " average CFG blocks per function.\n"
956 << " " << MaxCFGBlocksPerFunction
957 << " max CFG blocks per function.\n";
958
959 unsigned AvgUninitVariablesPerFunction = !NumUninitAnalysisFunctions ? 0
960 : NumUninitAnalysisVariables/NumUninitAnalysisFunctions;
961 unsigned AvgUninitBlockVisitsPerFunction = !NumUninitAnalysisFunctions ? 0
962 : NumUninitAnalysisBlockVisits/NumUninitAnalysisFunctions;
963 llvm::errs() << NumUninitAnalysisFunctions
964 << " functions analyzed for uninitialiazed variables\n"
965 << " " << NumUninitAnalysisVariables << " variables analyzed.\n"
966 << " " << AvgUninitVariablesPerFunction
967 << " average variables per function.\n"
968 << " " << MaxUninitAnalysisVariablesPerFunction
969 << " max variables per function.\n"
970 << " " << NumUninitAnalysisBlockVisits << " block visits.\n"
971 << " " << AvgUninitBlockVisitsPerFunction
972 << " average block visits per function.\n"
973 << " " << MaxUninitAnalysisBlockVisitsPerFunction
974 << " max block visits per function.\n";
Ted Kremenekdbdbaaf2010-03-20 21:06:02 +0000975}