BugReporter.cpp revision 0eb6c37dd4e4ad8fa2363003dea270f9fd6c2969
161f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek// BugReporter.cpp - Generate PathDiagnostics for Bugs ------------*- C++ -*--//
261f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//
361f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//                     The LLVM Compiler Infrastructure
461f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//
561f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek// This file is distributed under the University of Illinois Open Source
661f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek// License. See LICENSE.TXT for details.
761f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//
861f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//===----------------------------------------------------------------------===//
961f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//
1061f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//  This file defines BugReporter, a utility class for generating
116c07bdba93b095b66e2c8c82dd5ed458fa8285eaTed Kremenek//  PathDiagnostics.
1261f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//
1361f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek//===----------------------------------------------------------------------===//
1461f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek
159b663716449b618ba0390b1dbebc54fa8e971124Ted Kremenek#include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
169b663716449b618ba0390b1dbebc54fa8e971124Ted Kremenek#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
179b663716449b618ba0390b1dbebc54fa8e971124Ted Kremenek#include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
1861f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek#include "clang/AST/ASTContext.h"
19e41611aa2237d06a0ef61db4528fb2883a8defcdTed Kremenek#include "clang/Analysis/CFG.h"
20c35fb7d67d515659ad2325b4f6ec97c9fe64fb63Benjamin Kramer#include "clang/AST/DeclObjC.h"
2161f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek#include "clang/AST/Expr.h"
2200605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek#include "clang/AST/ParentMap.h"
2316f0049415ec596504891259e2a83e19871c0d52Chris Lattner#include "clang/AST/StmtObjC.h"
2416f0049415ec596504891259e2a83e19871c0d52Chris Lattner#include "clang/Basic/SourceManager.h"
2561f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek#include "clang/Analysis/ProgramPoint.h"
269b663716449b618ba0390b1dbebc54fa8e971124Ted Kremenek#include "clang/StaticAnalyzer/Core/BugReporter/PathDiagnostic.h"
27405674c26c52b05df0d833fae6bae818cd52bc32Chris Lattner#include "llvm/Support/raw_ostream.h"
28331b0ac44b9eb0ffcba66b4f3f3f9adb27c2434fTed Kremenek#include "llvm/ADT/DenseMap.h"
298fe83e1df954d72c0f4ffc15d20a5222ec151c21Benjamin Kramer#include "llvm/ADT/SmallString.h"
30cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek#include "llvm/ADT/STLExtras.h"
3100605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek#include "llvm/ADT/OwningPtr.h"
32802e02463b880f53a6e645bde78cc412481ce9e0Ted Kremenek#include "llvm/ADT/IntrusiveRefCntPtr.h"
3310aa55410bbecbb658ce14a5f801d50cf06d12dfTed Kremenek#include <queue>
3461f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek
3561f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenekusing namespace clang;
369ef6537a894c33003359b1f9b9676e9178e028b7Ted Kremenekusing namespace ento;
3761f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek
388966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed KremenekBugReporterVisitor::~BugReporterVisitor() {}
391b431023814196f87515a540ebcb9e9f1a9176a1Ted Kremenek
4099ba9e3bd70671f3441fb974895f226a83ce0e66David Blaikievoid BugReporterContext::anchor() {}
4199ba9e3bd70671f3441fb974895f226a83ce0e66David Blaikie
42cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek//===----------------------------------------------------------------------===//
433106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek// Helper routines for walking the ExplodedGraph and fetching statements.
44cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek//===----------------------------------------------------------------------===//
4561f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek
469c378f705405d37f49795d5e915989de774fe11fTed Kremenekstatic inline const Stmt *GetStmt(const ProgramPoint &P) {
47592362b46ad69db0db0988e7f9d8cbe647510bddTed Kremenek  if (const StmtPoint* SP = dyn_cast<StmtPoint>(&P))
48592362b46ad69db0db0988e7f9d8cbe647510bddTed Kremenek    return SP->getStmt();
499c378f705405d37f49795d5e915989de774fe11fTed Kremenek  else if (const BlockEdge *BE = dyn_cast<BlockEdge>(&P))
5061f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek    return BE->getSrc()->getTerminator();
51852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose  else if (const CallEnter *CE = dyn_cast<CallEnter>(&P))
52852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose    return CE->getCallExpr();
53852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose  else if (const CallExitEnd *CEE = dyn_cast<CallExitEnd>(&P))
54852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose    return CEE->getCalleeContext()->getCallSite();
551eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
56b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  return 0;
5761f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek}
5861f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek
59c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xustatic inline const ExplodedNode*
609c378f705405d37f49795d5e915989de774fe11fTed KremenekGetPredecessorNode(const ExplodedNode *N) {
61b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  return N->pred_empty() ? NULL : *(N->pred_begin());
62706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek}
63706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek
64c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xustatic inline const ExplodedNode*
659c378f705405d37f49795d5e915989de774fe11fTed KremenekGetSuccessorNode(const ExplodedNode *N) {
66b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  return N->succ_empty() ? NULL : *(N->succ_begin());
67bd7efa8ca27ed9676acf00abf31d5e1cd54651ccTed Kremenek}
682673c9ff76ebdb72b7f846e2ff1e0f4256372e19Ted Kremenek
699c378f705405d37f49795d5e915989de774fe11fTed Kremenekstatic const Stmt *GetPreviousStmt(const ExplodedNode *N) {
70b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  for (N = GetPredecessorNode(N); N; N = GetPredecessorNode(N))
715f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenek    if (const Stmt *S = GetStmt(N->getLocation()))
72b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek      return S;
731eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
74b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  return 0;
75b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek}
76b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek
779c378f705405d37f49795d5e915989de774fe11fTed Kremenekstatic const Stmt *GetNextStmt(const ExplodedNode *N) {
78b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  for (N = GetSuccessorNode(N); N; N = GetSuccessorNode(N))
795f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenek    if (const Stmt *S = GetStmt(N->getLocation())) {
80f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek      // Check if the statement is '?' or '&&'/'||'.  These are "merges",
81f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek      // not actual statement points.
82f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek      switch (S->getStmtClass()) {
83f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek        case Stmt::ChooseExprClass:
8456ca35d396d8692c384c785f9aeebcf22563fe1eJohn McCall        case Stmt::BinaryConditionalOperatorClass: continue;
85f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek        case Stmt::ConditionalOperatorClass: continue;
86f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek        case Stmt::BinaryOperatorClass: {
872de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall          BinaryOperatorKind Op = cast<BinaryOperator>(S)->getOpcode();
882de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall          if (Op == BO_LAnd || Op == BO_LOr)
89f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            continue;
90f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek          break;
91f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek        }
92f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek        default:
93f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek          break;
94f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek      }
95b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek      return S;
96f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek    }
971eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
98b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  return 0;
99bd7efa8ca27ed9676acf00abf31d5e1cd54651ccTed Kremenek}
100bd7efa8ca27ed9676acf00abf31d5e1cd54651ccTed Kremenek
1015f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenekstatic inline const Stmt*
1029c378f705405d37f49795d5e915989de774fe11fTed KremenekGetCurrentOrPreviousStmt(const ExplodedNode *N) {
1035f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenek  if (const Stmt *S = GetStmt(N->getLocation()))
104b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek    return S;
1051eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
106b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  return GetPreviousStmt(N);
107b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek}
1081eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1095f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenekstatic inline const Stmt*
1109c378f705405d37f49795d5e915989de774fe11fTed KremenekGetCurrentOrNextStmt(const ExplodedNode *N) {
1115f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenek  if (const Stmt *S = GetStmt(N->getLocation()))
112b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek    return S;
1131eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
114b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  return GetNextStmt(N);
1153148eb4a75f70f2636075c364d03104223f004d3Ted Kremenek}
1163148eb4a75f70f2636075c364d03104223f004d3Ted Kremenek
117b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek//===----------------------------------------------------------------------===//
118c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek// Diagnostic cleanup.
119c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek//===----------------------------------------------------------------------===//
120c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek
121c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek/// Recursively scan through a path and prune out calls and macros pieces
122c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek/// that aren't needed.  Return true if afterwards the path contains
123c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek/// "interesting stuff" which means it should be pruned from the parent path.
124c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenekstatic bool RemoveUneededCalls(PathPieces &pieces) {
125c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek  bool containsSomethingInteresting = false;
126c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek  const unsigned N = pieces.size();
127c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek
128c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek  for (unsigned i = 0 ; i < N ; ++i) {
129c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek    // Remove the front piece from the path.  If it is still something we
130c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek    // want to keep once we are done, we will push it back on the end.
131c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek    IntrusiveRefCntPtr<PathDiagnosticPiece> piece(pieces.front());
132c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek    pieces.pop_front();
133c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek
134725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek    switch (piece->getKind()) {
135725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek      case PathDiagnosticPiece::Call: {
136725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        PathDiagnosticCallPiece *call = cast<PathDiagnosticCallPiece>(piece);
137725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        // Recursively clean out the subclass.  Keep this call around if
138725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        // it contains any informative diagnostics.
139725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        if (!RemoveUneededCalls(call->path))
140725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek          continue;
141725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        containsSomethingInteresting = true;
142725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        break;
143725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek      }
144725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek      case PathDiagnosticPiece::Macro: {
145725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        PathDiagnosticMacroPiece *macro = cast<PathDiagnosticMacroPiece>(piece);
146725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        if (!RemoveUneededCalls(macro->subPieces))
147725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek          continue;
148c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek        containsSomethingInteresting = true;
149725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        break;
150725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek      }
151725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek      case PathDiagnosticPiece::Event: {
152725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        PathDiagnosticEventPiece *event = cast<PathDiagnosticEventPiece>(piece);
153725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        // We never throw away an event, but we do throw it away wholesale
154725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        // as part of a path if we throw the entire path away.
15576aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek        if (event->isPrunable())
15676aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek          continue;
15776aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek        containsSomethingInteresting = true;
158725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        break;
159725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek      }
160725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek      case PathDiagnosticPiece::ControlFlow:
161725167443808efdc39a99f4eb132a0ae64ac5118Ted Kremenek        break;
162c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek    }
163c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek
164c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek    pieces.push_back(piece);
165c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek  }
166c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek
167c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek  return containsSomethingInteresting;
168c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek}
169c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek
170c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek//===----------------------------------------------------------------------===//
1713106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek// PathDiagnosticBuilder and its associated routines and helper objects.
172b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek//===----------------------------------------------------------------------===//
173b479dadea93b93d3f0a335c3518b703f140b3f50Ted Kremenek
174c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xutypedef llvm::DenseMap<const ExplodedNode*,
175c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xuconst ExplodedNode*> NodeBackMap;
1767dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek
177babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremeneknamespace {
178ba5fb5a955c896815c439289fc51c03cf0635129Kovarththanan Rajaratnamclass NodeMapClosure : public BugReport::NodeResolver {
1797dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek  NodeBackMap& M;
1807dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenekpublic:
1817dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek  NodeMapClosure(NodeBackMap *m) : M(*m) {}
1827dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek  ~NodeMapClosure() {}
1831eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1849c378f705405d37f49795d5e915989de774fe11fTed Kremenek  const ExplodedNode *getOriginalNode(const ExplodedNode *N) {
1857dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek    NodeBackMap::iterator I = M.find(N);
1867dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek    return I == M.end() ? 0 : I->second;
1877dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek  }
1887dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek};
1891eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
190ba5fb5a955c896815c439289fc51c03cf0635129Kovarththanan Rajaratnamclass PathDiagnosticBuilder : public BugReporterContext {
1917dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek  BugReport *R;
192ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie  PathDiagnosticConsumer *PDC;
1936f42b62b6194f53bcbc349f5d17388e1936535d7Dylan Noblesmith  OwningPtr<ParentMap> PM;
1947dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek  NodeMapClosure NMC;
1951eb4433ac451dc16f4133a88af2d002ac26c58efMike Stumppublic:
19659950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek  const LocationContext *LC;
19759950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek
1988966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek  PathDiagnosticBuilder(GRBugReporter &br,
1991eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump                        BugReport *r, NodeBackMap *Backmap,
200ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie                        PathDiagnosticConsumer *pdc)
2018966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek    : BugReporterContext(br),
20259950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek      R(r), PDC(pdc), NMC(Backmap), LC(r->getErrorNode()->getLocationContext())
20359950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek  {}
2041eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2059c378f705405d37f49795d5e915989de774fe11fTed Kremenek  PathDiagnosticLocation ExecutionContinues(const ExplodedNode *N);
2061eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2079c378f705405d37f49795d5e915989de774fe11fTed Kremenek  PathDiagnosticLocation ExecutionContinues(llvm::raw_string_ostream &os,
2089c378f705405d37f49795d5e915989de774fe11fTed Kremenek                                            const ExplodedNode *N);
2091eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2108e6431adab313e283a992698f6fc7afe62420999Anna Zaks  BugReport *getBugReport() { return R; }
2118e6431adab313e283a992698f6fc7afe62420999Anna Zaks
212212f6d3b5fb3fa55ba1e40671cfc336430abc8ddTom Care  Decl const &getCodeDecl() { return R->getErrorNode()->getCodeDecl(); }
21359950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek
21459950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek  ParentMap& getParentMap() { return LC->getParentMap(); }
2151eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
216c3f83ad7adaff2623986aa1c3e57833babd03d50Ted Kremenek  const Stmt *getParent(const Stmt *S) {
217c3f83ad7adaff2623986aa1c3e57833babd03d50Ted Kremenek    return getParentMap().getParent(S);
218c3f83ad7adaff2623986aa1c3e57833babd03d50Ted Kremenek  }
2191eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2208966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek  virtual NodeMapClosure& getNodeResolver() { return NMC; }
2217297134f128423fce2e88f92421ed135bded7d4eDouglas Gregor
222d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek  PathDiagnosticLocation getEnclosingStmtLocation(const Stmt *S);
2231eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
224ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie  PathDiagnosticConsumer::PathGenerationScheme getGenerationScheme() const {
225ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie    return PDC ? PDC->getGenerationScheme() : PathDiagnosticConsumer::Extensive;
2267dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek  }
2277dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek
228babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek  bool supportsLogicalOpControlFlow() const {
229babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek    return PDC ? PDC->supportsLogicalOpControlFlow() : true;
2301eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  }
231babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek};
232babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek} // end anonymous namespace
233babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek
23400605e0f011ed99225a22026344c5fc1b3212cd7Ted KremenekPathDiagnosticLocation
2359c378f705405d37f49795d5e915989de774fe11fTed KremenekPathDiagnosticBuilder::ExecutionContinues(const ExplodedNode *N) {
2365f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenek  if (const Stmt *S = GetNextStmt(N))
23759950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek    return PathDiagnosticLocation(S, getSourceManager(), LC);
23800605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek
2390cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks  return PathDiagnosticLocation::createDeclEnd(N->getLocationContext(),
2400cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks                                               getSourceManager());
241082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek}
2421eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
24300605e0f011ed99225a22026344c5fc1b3212cd7Ted KremenekPathDiagnosticLocation
2449c378f705405d37f49795d5e915989de774fe11fTed KremenekPathDiagnosticBuilder::ExecutionContinues(llvm::raw_string_ostream &os,
2459c378f705405d37f49795d5e915989de774fe11fTed Kremenek                                          const ExplodedNode *N) {
246babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek
247143ca222583a4a355fdc89af852deef287499300Ted Kremenek  // Slow, but probably doesn't matter.
248b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek  if (os.str().empty())
249b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek    os << ' ';
2501eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
25100605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek  const PathDiagnosticLocation &Loc = ExecutionContinues(N);
2521eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
25300605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek  if (Loc.asStmt())
254b697b100f2354ac84b7a0785cab9481e8bfdcf23Ted Kremenek    os << "Execution continues on line "
255642116259e8df6286063a17361c20e95b5017a0aChandler Carruth       << getSourceManager().getExpansionLineNumber(Loc.asLocation())
2568966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek       << '.';
2574f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek  else {
2584f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek    os << "Execution jumps to the end of the ";
2594f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek    const Decl *D = N->getLocationContext()->getDecl();
2604f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek    if (isa<ObjCMethodDecl>(D))
2614f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek      os << "method";
2624f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek    else if (isa<FunctionDecl>(D))
2634f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek      os << "function";
2644f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek    else {
2654f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek      assert(isa<BlockDecl>(D));
2664f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek      os << "anonymous block";
2674f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek    }
2684f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek    os << '.';
2694f1db537722e8d46146cd95f5962a6dcf6e44e54Ted Kremenek  }
2701eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
271082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek  return Loc;
272143ca222583a4a355fdc89af852deef287499300Ted Kremenek}
273143ca222583a4a355fdc89af852deef287499300Ted Kremenek
274ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenekstatic bool IsNested(const Stmt *S, ParentMap &PM) {
275ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek  if (isa<Expr>(S) && PM.isConsumedExpr(cast<Expr>(S)))
276ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek    return true;
2771eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
278ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek  const Stmt *Parent = PM.getParentIgnoreParens(S);
2791eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
280ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek  if (Parent)
281ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek    switch (Parent->getStmtClass()) {
282ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek      case Stmt::ForStmtClass:
283ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek      case Stmt::DoStmtClass:
284ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek      case Stmt::WhileStmtClass:
285ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek        return true;
286ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek      default:
287ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek        break;
288ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek    }
2891eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2901eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  return false;
291ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek}
292ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek
293d8c938b0f6b7a7156181be575239e4c6e15a2adbTed KremenekPathDiagnosticLocation
294d8c938b0f6b7a7156181be575239e4c6e15a2adbTed KremenekPathDiagnosticBuilder::getEnclosingStmtLocation(const Stmt *S) {
2959c378f705405d37f49795d5e915989de774fe11fTed Kremenek  assert(S && "Null Stmt *passed to getEnclosingStmtLocation");
2961eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  ParentMap &P = getParentMap();
2978966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek  SourceManager &SMgr = getSourceManager();
298e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek
299ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek  while (IsNested(S, P)) {
3008c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek    const Stmt *Parent = P.getParentIgnoreParens(S);
3011eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
302af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek    if (!Parent)
303af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      break;
3041eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
305af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek    switch (Parent->getStmtClass()) {
3065fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek      case Stmt::BinaryOperatorClass: {
3075fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek        const BinaryOperator *B = cast<BinaryOperator>(Parent);
3085fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek        if (B->isLogicalOp())
309220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(S, SMgr, LC);
3105fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek        break;
3111eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump      }
312af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      case Stmt::CompoundStmtClass:
313af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      case Stmt::StmtExprClass:
314220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks        return PathDiagnosticLocation(S, SMgr, LC);
3151d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek      case Stmt::ChooseExprClass:
3161d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek        // Similar to '?' if we are referring to condition, just have the edge
3171d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek        // point to the entire choose expression.
3181d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek        if (cast<ChooseExpr>(Parent)->getCond() == S)
319220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(Parent, SMgr, LC);
3201d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek        else
321220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(S, SMgr, LC);
32256ca35d396d8692c384c785f9aeebcf22563fe1eJohn McCall      case Stmt::BinaryConditionalOperatorClass:
3231d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek      case Stmt::ConditionalOperatorClass:
3241d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek        // For '?', if we are referring to condition, just have the edge point
3251d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek        // to the entire '?' expression.
32656ca35d396d8692c384c785f9aeebcf22563fe1eJohn McCall        if (cast<AbstractConditionalOperator>(Parent)->getCond() == S)
327220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(Parent, SMgr, LC);
3281d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek        else
329220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(S, SMgr, LC);
330af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      case Stmt::DoStmtClass:
331220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(S, SMgr, LC);
332af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      case Stmt::ForStmtClass:
333af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek        if (cast<ForStmt>(Parent)->getBody() == S)
334220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(S, SMgr, LC);
3351eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump        break;
336af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      case Stmt::IfStmtClass:
337af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek        if (cast<IfStmt>(Parent)->getCond() != S)
338220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(S, SMgr, LC);
3398bd4d037d837e7922c3661d6158229da58a03887Ted Kremenek        break;
340af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      case Stmt::ObjCForCollectionStmtClass:
341af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek        if (cast<ObjCForCollectionStmt>(Parent)->getBody() == S)
342220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(S, SMgr, LC);
343af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek        break;
344af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      case Stmt::WhileStmtClass:
345af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek        if (cast<WhileStmt>(Parent)->getCond() != S)
346220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(S, SMgr, LC);
347af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek        break;
348af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek      default:
349af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek        break;
350af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek    }
351af3e3d54e990b385e5a653d2994d7d41427a13b8Ted Kremenek
352d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek    S = Parent;
353d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek  }
3541eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
355d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek  assert(S && "Cannot have null Stmt for PathDiagnosticLocation");
356e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek
357e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek  // Special case: DeclStmts can appear in for statement declarations, in which
358e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek  //  case the ForStmt is the context.
359e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek  if (isa<DeclStmt>(S)) {
360e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek    if (const Stmt *Parent = P.getParent(S)) {
361e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek      switch (Parent->getStmtClass()) {
362e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek        case Stmt::ForStmtClass:
363e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek        case Stmt::ObjCForCollectionStmtClass:
364220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks          return PathDiagnosticLocation(Parent, SMgr, LC);
365e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek        default:
366e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek          break;
3671eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump      }
3681eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump    }
369e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek  }
370e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek  else if (isa<BinaryOperator>(S)) {
371e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek    // Special case: the binary operator represents the initialization
372e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek    // code in a for statement (this can happen when the variable being
373e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek    // initialized is an old variable.
374e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek    if (const ForStmt *FS =
375e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek          dyn_cast_or_null<ForStmt>(P.getParentIgnoreParens(S))) {
376e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek      if (FS->getInit() == S)
377220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks        return PathDiagnosticLocation(FS, SMgr, LC);
378e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek    }
379e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek  }
380e88a170da9219ef6a62d2182560c4de2ebffbd59Ted Kremenek
381220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks  return PathDiagnosticLocation(S, SMgr, LC);
382d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek}
383d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek
384cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek//===----------------------------------------------------------------------===//
3853106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek// "Minimal" path diagnostic generation algorithm.
386cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek//===----------------------------------------------------------------------===//
38756a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zakstypedef std::pair<PathDiagnosticCallPiece*, const ExplodedNode*> StackDiagPair;
38856a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zakstypedef SmallVector<StackDiagPair, 6> StackDiagVector;
38956a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks
390368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaksstatic void updateStackPiecesWithMessage(PathDiagnosticPiece *P,
39156a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks                                         StackDiagVector &CallStack) {
392368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks  // If the piece contains a special message, add it to all the call
393368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks  // pieces on the active stack.
394368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks  if (PathDiagnosticEventPiece *ep =
395368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        dyn_cast<PathDiagnosticEventPiece>(P)) {
396368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks
39756a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks    if (ep->hasCallStackHint())
39856a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks      for (StackDiagVector::iterator I = CallStack.begin(),
39956a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks                                     E = CallStack.end(); I != E; ++I) {
40056a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks        PathDiagnosticCallPiece *CP = I->first;
40156a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks        const ExplodedNode *N = I->second;
4028fe4525680ce72e90cee3e58b5654e3ae955447fNAKAMURA Takumi        std::string stackMsg = ep->getCallStackMessage(N);
40356a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks
404368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        // The last message on the path to final bug is the most important
405368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        // one. Since we traverse the path backwards, do not add the message
406368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        // if one has been previously added.
40756a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks        if  (!CP->hasCallStackMessage())
40856a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks          CP->setCallStackMessage(stackMsg);
40956a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks      }
410368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks  }
411368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks}
412cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek
41377d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenekstatic void CompactPathDiagnostic(PathPieces &path, const SourceManager& SM);
41414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
4153106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenekstatic void GenerateMinimalPathDiagnostic(PathDiagnostic& PD,
4163106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek                                          PathDiagnosticBuilder &PDB,
4173bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose                                          const ExplodedNode *N,
4183bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose                                      ArrayRef<BugReporterVisitor *> visitors) {
4198966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek
4203106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  SourceManager& SMgr = PDB.getSourceManager();
42159950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek  const LocationContext *LC = PDB.LC;
4229c378f705405d37f49795d5e915989de774fe11fTed Kremenek  const ExplodedNode *NextNode = N->pred_empty()
4233106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek                                        ? NULL : *(N->pred_begin());
424368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks
42556a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks  StackDiagVector CallStack;
426368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks
4273106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  while (NextNode) {
4281eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump    N = NextNode;
42959950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek    PDB.LC = N->getLocationContext();
4303106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    NextNode = GetPredecessorNode(N);
4311eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
43261f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek    ProgramPoint P = N->getLocation();
4332042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek
4340b3ade86a1c60cf0c7b56aa238aff458eb7f5974Anna Zaks    if (const CallExitEnd *CE = dyn_cast<CallExitEnd>(&P)) {
4352042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      PathDiagnosticCallPiece *C =
4362042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek        PathDiagnosticCallPiece::construct(N, *CE, SMgr);
4372042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      PD.getActivePath().push_front(C);
4382042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      PD.pushActivePath(&C->path);
43956a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks      CallStack.push_back(StackDiagPair(C, N));
4402042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      continue;
4412042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek    }
4422042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek
4432042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek    if (const CallEnter *CE = dyn_cast<CallEnter>(&P)) {
444183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose      // Flush all locations, and pop the active path.
445183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose      bool VisitedEntireCall = PD.isWithinCall();
4462042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      PD.popActivePath();
447183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose
448183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose      // Either we just added a bunch of stuff to the top-level path, or
449183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose      // we have a previous CallExitEnd.  If the former, it means that the
4502042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      // path terminated within a function call.  We must then take the
4512042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      // current contents of the active path and place it within
4522042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      // a new PathDiagnosticCallPiece.
453183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose      PathDiagnosticCallPiece *C;
454183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose      if (VisitedEntireCall) {
455183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        C = cast<PathDiagnosticCallPiece>(PD.getActivePath().front());
456183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose      } else {
4579373937945e1e075dfa08169eaccc1ad0b31f699Anna Zaks        const Decl *Caller = CE->getLocationContext()->getDecl();
4589373937945e1e075dfa08169eaccc1ad0b31f699Anna Zaks        C = PathDiagnosticCallPiece::construct(PD.getActivePath(), Caller);
4599373937945e1e075dfa08169eaccc1ad0b31f699Anna Zaks      }
460183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose
4612042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      C->setCallee(*CE, SMgr);
462368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks      if (!CallStack.empty()) {
46356a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks        assert(CallStack.back().first == C);
464368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        CallStack.pop_back();
465368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks      }
4662042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      continue;
4672042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek    }
4681eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
4699c378f705405d37f49795d5e915989de774fe11fTed Kremenek    if (const BlockEdge *BE = dyn_cast<BlockEdge>(&P)) {
4709c378f705405d37f49795d5e915989de774fe11fTed Kremenek      const CFGBlock *Src = BE->getSrc();
4719c378f705405d37f49795d5e915989de774fe11fTed Kremenek      const CFGBlock *Dst = BE->getDst();
4729c378f705405d37f49795d5e915989de774fe11fTed Kremenek      const Stmt *T = Src->getTerminator();
4731eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
47461f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek      if (!T)
47561f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek        continue;
4761eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
477590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks      PathDiagnosticLocation Start =
478590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks        PathDiagnosticLocation::createBegin(T, SMgr,
479590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks                                                N->getLocationContext());
4801eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
48161f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek      switch (T->getStmtClass()) {
48261f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek        default:
48361f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek          break;
4841eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
48561f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek        case Stmt::GotoStmtClass:
4861eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump        case Stmt::IndirectGotoStmtClass: {
4879c378f705405d37f49795d5e915989de774fe11fTed Kremenek          const Stmt *S = GetNextStmt(N);
4881eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
48961f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek          if (!S)
49061f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek            continue;
4911eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
492297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek          std::string sbuf;
4931eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump          llvm::raw_string_ostream os(sbuf);
494d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek          const PathDiagnosticLocation &End = PDB.getEnclosingStmtLocation(S);
4951eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
49600605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek          os << "Control jumps to line "
497642116259e8df6286063a17361c20e95b5017a0aChandler Carruth          << End.asLocation().getExpansionLineNumber();
4982042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek          PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
499802e02463b880f53a6e645bde78cc412481ce9e0Ted Kremenek                                                                os.str()));
50061f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek          break;
50161f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek        }
5021eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5031eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump        case Stmt::SwitchStmtClass: {
50461f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek          // Figure out what case arm we took.
505297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek          std::string sbuf;
506297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek          llvm::raw_string_ostream os(sbuf);
5071eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5089c378f705405d37f49795d5e915989de774fe11fTed Kremenek          if (const Stmt *S = Dst->getLabel()) {
509220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks            PathDiagnosticLocation End(S, SMgr, LC);
5101eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5115a429954aedd8a830a70fcea2cb51561673e4fc3Ted Kremenek            switch (S->getStmtClass()) {
512082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek              default:
513082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                os << "No cases match in the switch statement. "
5143106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek                "Control jumps to line "
515642116259e8df6286063a17361c20e95b5017a0aChandler Carruth                << End.asLocation().getExpansionLineNumber();
516082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                break;
517082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek              case Stmt::DefaultStmtClass:
518082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                os << "Control jumps to the 'default' case at line "
519642116259e8df6286063a17361c20e95b5017a0aChandler Carruth                << End.asLocation().getExpansionLineNumber();
520082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                break;
5211eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
522082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek              case Stmt::CaseStmtClass: {
5231eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump                os << "Control jumps to 'case ";
5249c378f705405d37f49795d5e915989de774fe11fTed Kremenek                const CaseStmt *Case = cast<CaseStmt>(S);
5259c378f705405d37f49795d5e915989de774fe11fTed Kremenek                const Expr *LHS = Case->getLHS()->IgnoreParenCasts();
5261eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5271eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump                // Determine if it is an enum.
528082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                bool GetRawInt = true;
5291eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5309c378f705405d37f49795d5e915989de774fe11fTed Kremenek                if (const DeclRefExpr *DR = dyn_cast<DeclRefExpr>(LHS)) {
531082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                  // FIXME: Maybe this should be an assertion.  Are there cases
532082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                  // were it is not an EnumConstantDecl?
5339c378f705405d37f49795d5e915989de774fe11fTed Kremenek                  const EnumConstantDecl *D =
53403509aea098772644bf4662dc1c88634818ceeccZhongxing Xu                    dyn_cast<EnumConstantDecl>(DR->getDecl());
5351eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
536082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                  if (D) {
537082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                    GetRawInt = false;
538b8989f27f116ff2400e92a52c067a69846119eb5Benjamin Kramer                    os << *D;
539082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                  }
5405a429954aedd8a830a70fcea2cb51561673e4fc3Ted Kremenek                }
5419ec64d6e2fe575b297e1eaa5051efc2983373e25Eli Friedman
5429ec64d6e2fe575b297e1eaa5051efc2983373e25Eli Friedman                if (GetRawInt)
543a6b8b2c09610b8bc4330e948ece8b940c2386406Richard Smith                  os << LHS->EvaluateKnownConstInt(PDB.getASTContext());
5449ec64d6e2fe575b297e1eaa5051efc2983373e25Eli Friedman
54500605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek                os << ":'  at line "
546642116259e8df6286063a17361c20e95b5017a0aChandler Carruth                << End.asLocation().getExpansionLineNumber();
547082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                break;
548082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek              }
54961f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek            }
5502042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek            PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
55100605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek                                                             os.str()));
55261f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek          }
553567839273dfc395a459cbe77d5f8d128b9ae85bfTed Kremenek          else {
554c3517ebb48a51e146badc08bc3684520c41da6c8Ted Kremenek            os << "'Default' branch taken. ";
5551eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump            const PathDiagnosticLocation &End = PDB.ExecutionContinues(os, N);
5562042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek            PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
55700605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek                                                             os.str()));
558567839273dfc395a459cbe77d5f8d128b9ae85bfTed Kremenek          }
5591eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
56061f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek          break;
56161f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek        }
5621eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5632673c9ff76ebdb72b7f846e2ff1e0f4256372e19Ted Kremenek        case Stmt::BreakStmtClass:
5642673c9ff76ebdb72b7f846e2ff1e0f4256372e19Ted Kremenek        case Stmt::ContinueStmtClass: {
565297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek          std::string sbuf;
566297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek          llvm::raw_string_ostream os(sbuf);
56700605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek          PathDiagnosticLocation End = PDB.ExecutionContinues(os, N);
5682042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek          PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
569082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                                                           os.str()));
5702673c9ff76ebdb72b7f846e2ff1e0f4256372e19Ted Kremenek          break;
5712673c9ff76ebdb72b7f846e2ff1e0f4256372e19Ted Kremenek        }
5721eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5733106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          // Determine control-flow for ternary '?'.
57456ca35d396d8692c384c785f9aeebcf22563fe1eJohn McCall        case Stmt::BinaryConditionalOperatorClass:
575706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek        case Stmt::ConditionalOperatorClass: {
576297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek          std::string sbuf;
577297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek          llvm::raw_string_ostream os(sbuf);
5781d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek          os << "'?' condition is ";
5791eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
580706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek          if (*(Src->succ_begin()+1) == Dst)
581082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek            os << "false";
582706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek          else
583082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek            os << "true";
5841eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
58500605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek          PathDiagnosticLocation End = PDB.ExecutionContinues(N);
5861eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5871d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek          if (const Stmt *S = End.asStmt())
5881d9a23ac34fee297e28361653a72c519cb04a854Ted Kremenek            End = PDB.getEnclosingStmtLocation(S);
5891eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5902042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek          PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
591babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek                                                           os.str()));
592babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek          break;
593babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek        }
5941eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
5953106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          // Determine control-flow for short-circuited '&&' and '||'.
596babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek        case Stmt::BinaryOperatorClass: {
597babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek          if (!PDB.supportsLogicalOpControlFlow())
598babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek            break;
5991eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
60003509aea098772644bf4662dc1c88634818ceeccZhongxing Xu          const BinaryOperator *B = cast<BinaryOperator>(T);
601babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek          std::string sbuf;
602babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek          llvm::raw_string_ostream os(sbuf);
603babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek          os << "Left side of '";
6041eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6052de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall          if (B->getOpcode() == BO_LAnd) {
606f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            os << "&&" << "' is ";
6071eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
608f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            if (*(Src->succ_begin()+1) == Dst) {
609f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek              os << "false";
610220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks              PathDiagnosticLocation End(B->getLHS(), SMgr, LC);
6110cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks              PathDiagnosticLocation Start =
6120cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks                PathDiagnosticLocation::createOperatorLoc(B, SMgr);
6132042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek              PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
614f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek                                                               os.str()));
6151eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump            }
616f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            else {
617f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek              os << "true";
618220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks              PathDiagnosticLocation Start(B->getLHS(), SMgr, LC);
619f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek              PathDiagnosticLocation End = PDB.ExecutionContinues(N);
6202042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek              PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
621f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek                                                               os.str()));
6221eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump            }
623babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek          }
624babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek          else {
6252de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall            assert(B->getOpcode() == BO_LOr);
626f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            os << "||" << "' is ";
6271eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
628f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            if (*(Src->succ_begin()+1) == Dst) {
629f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek              os << "false";
630220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks              PathDiagnosticLocation Start(B->getLHS(), SMgr, LC);
631f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek              PathDiagnosticLocation End = PDB.ExecutionContinues(N);
6322042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek              PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
6331eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump                                                               os.str()));
634f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            }
635f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            else {
636f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek              os << "true";
637220ac8c175cb1bf9b18d82eefe036995d7a2164dAnna Zaks              PathDiagnosticLocation End(B->getLHS(), SMgr, LC);
6380cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks              PathDiagnosticLocation Start =
6390cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks                PathDiagnosticLocation::createOperatorLoc(B, SMgr);
6402042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek              PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
6411eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump                                                               os.str()));
642f5ab8e6eb4b1c14cb3a0f69aad526192794c95ceTed Kremenek            }
643babdd7b56d02e2b6924a2c93b061d6a48bb5f0caTed Kremenek          }
6441eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
645706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek          break;
646706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek        }
6471eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6481eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump        case Stmt::DoStmtClass:  {
649706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek          if (*(Src->succ_begin()) == Dst) {
650297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek            std::string sbuf;
651297308eda369de400e6ea7057dffeadd9f92e385Ted Kremenek            llvm::raw_string_ostream os(sbuf);
6521eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
653c3517ebb48a51e146badc08bc3684520c41da6c8Ted Kremenek            os << "Loop condition is true. ";
654d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek            PathDiagnosticLocation End = PDB.ExecutionContinues(os, N);
6551eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
656d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek            if (const Stmt *S = End.asStmt())
657d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek              End = PDB.getEnclosingStmtLocation(S);
6581eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6592042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek            PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
660082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek                                                             os.str()));
661082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek          }
662082cb8d7bd9bdb3fe58e8e1a2897c79c4ebcc3a7Ted Kremenek          else {
66300605e0f011ed99225a22026344c5fc1b3212cd7Ted Kremenek            PathDiagnosticLocation End = PDB.ExecutionContinues(N);
6641eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
665d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek            if (const Stmt *S = End.asStmt())
666d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek              End = PDB.getEnclosingStmtLocation(S);
6671eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6682042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek            PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
6693106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek                              "Loop condition is false.  Exiting loop"));
6703106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          }
6711eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6723106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          break;
6733106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        }
6741eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6753106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        case Stmt::WhileStmtClass:
6761eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump        case Stmt::ForStmtClass: {
6773106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          if (*(Src->succ_begin()+1) == Dst) {
6783106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek            std::string sbuf;
6793106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek            llvm::raw_string_ostream os(sbuf);
6801eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6813106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek            os << "Loop condition is false. ";
6823106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek            PathDiagnosticLocation End = PDB.ExecutionContinues(os, N);
6833106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek            if (const Stmt *S = End.asStmt())
6843106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek              End = PDB.getEnclosingStmtLocation(S);
6851eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6862042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek            PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
6873106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek                                                             os.str()));
6883106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          }
6893106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          else {
6903106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek            PathDiagnosticLocation End = PDB.ExecutionContinues(N);
6913106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek            if (const Stmt *S = End.asStmt())
6923106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek              End = PDB.getEnclosingStmtLocation(S);
6931eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6942042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek            PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
6955fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek                            "Loop condition is true.  Entering loop body"));
6963106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          }
6971eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
6983106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          break;
6993106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        }
7001eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7013106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        case Stmt::IfStmtClass: {
7023106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          PathDiagnosticLocation End = PDB.ExecutionContinues(N);
7031eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7043106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          if (const Stmt *S = End.asStmt())
7053106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek            End = PDB.getEnclosingStmtLocation(S);
7061eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7073106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          if (*(Src->succ_begin()+1) == Dst)
7082042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek            PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
7095fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek                                                        "Taking false branch"));
7101eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump          else
7112042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek            PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(Start, End,
7125fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek                                                         "Taking true branch"));
7131eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7143106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          break;
7153106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        }
7163106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      }
7173106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    }
7181eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
719dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek    if (NextNode) {
7208e6431adab313e283a992698f6fc7afe62420999Anna Zaks      // Add diagnostic pieces from custom visitors.
7218e6431adab313e283a992698f6fc7afe62420999Anna Zaks      BugReport *R = PDB.getBugReport();
7223bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose      for (ArrayRef<BugReporterVisitor *>::iterator I = visitors.begin(),
7233bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose                                                    E = visitors.end();
7243bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose           I != E; ++I) {
725368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        if (PathDiagnosticPiece *p = (*I)->VisitNode(N, NextNode, PDB, *R)) {
7262042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek          PD.getActivePath().push_front(p);
727368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks          updateStackPiecesWithMessage(p, CallStack);
728368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        }
729dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek      }
7308966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek    }
7313106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  }
7321eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
73314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  // After constructing the full PathDiagnostic, do a pass over it to compact
73414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  // PathDiagnosticPieces that occur within a macro.
73577d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek  CompactPathDiagnostic(PD.getMutablePieces(), PDB.getSourceManager());
7363106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek}
7373106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
7383106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek//===----------------------------------------------------------------------===//
7395fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek// "Extensive" PathDiagnostic generation.
7405fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek//===----------------------------------------------------------------------===//
7415fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek
7425fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenekstatic bool IsControlFlowExpr(const Stmt *S) {
7435fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek  const Expr *E = dyn_cast<Expr>(S);
7441eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7455fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek  if (!E)
7465fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek    return false;
7471eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7481eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  E = E->IgnoreParenCasts();
7491eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
75056ca35d396d8692c384c785f9aeebcf22563fe1eJohn McCall  if (isa<AbstractConditionalOperator>(E))
7515fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek    return true;
7521eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7535fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek  if (const BinaryOperator *B = dyn_cast<BinaryOperator>(E))
7545fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek    if (B->isLogicalOp())
7555fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek      return true;
7561eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7571eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  return false;
7585fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek}
7595fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek
76014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremeneknamespace {
761ba5fb5a955c896815c439289fc51c03cf0635129Kovarththanan Rajaratnamclass ContextLocation : public PathDiagnosticLocation {
7628f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek  bool IsDead;
7638f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenekpublic:
7648f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek  ContextLocation(const PathDiagnosticLocation &L, bool isdead = false)
7658f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek    : PathDiagnosticLocation(L), IsDead(isdead) {}
7661eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7671eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  void markDead() { IsDead = true; }
7688f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek  bool isDead() const { return IsDead; }
7698f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek};
7701eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
771ba5fb5a955c896815c439289fc51c03cf0635129Kovarththanan Rajaratnamclass EdgeBuilder {
7728f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek  std::vector<ContextLocation> CLocs;
7738f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek  typedef std::vector<ContextLocation>::iterator iterator;
77414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  PathDiagnostic &PD;
77514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  PathDiagnosticBuilder &PDB;
77614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  PathDiagnosticLocation PrevLoc;
7771eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7788f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek  bool IsConsumedExpr(const PathDiagnosticLocation &L);
7791eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
78014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  bool containsLocation(const PathDiagnosticLocation &Container,
78114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek                        const PathDiagnosticLocation &Containee);
7821eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
78314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  PathDiagnosticLocation getContextLocation(const PathDiagnosticLocation &L);
7841eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
7859650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek  PathDiagnosticLocation cleanUpLocation(PathDiagnosticLocation L,
7869650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek                                         bool firstCharOnly = false) {
7878c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek    if (const Stmt *S = L.asStmt()) {
7889650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek      const Stmt *Original = S;
7898c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek      while (1) {
7908c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek        // Adjust the location for some expressions that are best referenced
7918c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek        // by one of their subexpressions.
7929650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek        switch (S->getStmtClass()) {
7939650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek          default:
7949650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            break;
7959650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek          case Stmt::ParenExprClass:
796f111d935722ed488144600cea5ed03a6b5069e8fPeter Collingbourne          case Stmt::GenericSelectionExprClass:
797f111d935722ed488144600cea5ed03a6b5069e8fPeter Collingbourne            S = cast<Expr>(S)->IgnoreParens();
7989650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            firstCharOnly = true;
7999650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            continue;
80056ca35d396d8692c384c785f9aeebcf22563fe1eJohn McCall          case Stmt::BinaryConditionalOperatorClass:
8019650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek          case Stmt::ConditionalOperatorClass:
80256ca35d396d8692c384c785f9aeebcf22563fe1eJohn McCall            S = cast<AbstractConditionalOperator>(S)->getCond();
8039650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            firstCharOnly = true;
8049650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            continue;
8059650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek          case Stmt::ChooseExprClass:
8069650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            S = cast<ChooseExpr>(S)->getCond();
8079650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            firstCharOnly = true;
8089650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            continue;
8099650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek          case Stmt::BinaryOperatorClass:
8109650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            S = cast<BinaryOperator>(S)->getLHS();
8119650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            firstCharOnly = true;
8129650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek            continue;
8139650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek        }
8141eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
8159650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek        break;
8168c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek      }
8171eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
8189650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek      if (S != Original)
81959950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek        L = PathDiagnosticLocation(S, L.getManager(), PDB.LC);
8208c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek    }
8211eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
8229650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek    if (firstCharOnly)
8231531bb0c69d9afff6a6434e4cadf345eb628b287Anna Zaks      L  = PathDiagnosticLocation::createSingleLocation(L);
8249650cf3b08fcd66caae39bd5a1915c3eac735095Ted Kremenek
8258c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek    return L;
8268c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek  }
8271eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
82814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  void popLocation() {
8298f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek    if (!CLocs.back().isDead() && CLocs.back().asLocation().isFileID()) {
8305c7168cbe8a0eabd05e0e12e469090d2cabd27c0Ted Kremenek      // For contexts, we only one the first character as the range.
83107c015cbcee31f72e2d320ba2c713c046bed42faTed Kremenek      rawAddEdge(cleanUpLocation(CLocs.back(), true));
8325c7168cbe8a0eabd05e0e12e469090d2cabd27c0Ted Kremenek    }
83314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    CLocs.pop_back();
83414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  }
8351eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
83614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenekpublic:
83714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  EdgeBuilder(PathDiagnostic &pd, PathDiagnosticBuilder &pdb)
83814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    : PD(pd), PDB(pdb) {
8391eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
840a301a6773db085575ac51e3c966858180390c25bTed Kremenek      // If the PathDiagnostic already has pieces, add the enclosing statement
841a301a6773db085575ac51e3c966858180390c25bTed Kremenek      // of the first piece as a context as well.
842802e02463b880f53a6e645bde78cc412481ce9e0Ted Kremenek      if (!PD.path.empty()) {
843802e02463b880f53a6e645bde78cc412481ce9e0Ted Kremenek        PrevLoc = (*PD.path.begin())->getLocation();
8441eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
84514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek        if (const Stmt *S = PrevLoc.asStmt())
846e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek          addExtendedContext(PDB.getEnclosingStmtLocation(S).asStmt());
84714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      }
84814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  }
84914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
85014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  ~EdgeBuilder() {
85114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    while (!CLocs.empty()) popLocation();
8520cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks
853a301a6773db085575ac51e3c966858180390c25bTed Kremenek    // Finally, add an initial edge from the start location of the first
854a301a6773db085575ac51e3c966858180390c25bTed Kremenek    // statement (if it doesn't already exist).
8550cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks    PathDiagnosticLocation L = PathDiagnosticLocation::createDeclBegin(
85659950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek                                                       PDB.LC,
8570cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks                                                       PDB.getSourceManager());
8580cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks    if (L.isValid())
8590cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks      rawAddEdge(L);
86014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  }
86114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
8625de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek  void flushLocations() {
8635de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek    while (!CLocs.empty())
8645de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek      popLocation();
8655de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek    PrevLoc = PathDiagnosticLocation();
8665de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek  }
8675de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek
86814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  void addEdge(PathDiagnosticLocation NewLoc, bool alwaysAdd = false);
8691eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
8708bd4d037d837e7922c3661d6158229da58a03887Ted Kremenek  void rawAddEdge(PathDiagnosticLocation NewLoc);
8711eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
87214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  void addContext(const Stmt *S);
873183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose  void addContext(const PathDiagnosticLocation &L);
874e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek  void addExtendedContext(const Stmt *S);
8751eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump};
87614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek} // end anonymous namespace
87714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
87814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
87914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted KremenekPathDiagnosticLocation
88014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted KremenekEdgeBuilder::getContextLocation(const PathDiagnosticLocation &L) {
88114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  if (const Stmt *S = L.asStmt()) {
88214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    if (IsControlFlowExpr(S))
88314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      return L;
8841eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
8851eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump    return PDB.getEnclosingStmtLocation(S);
88614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  }
8871eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
88814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  return L;
88914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek}
89014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
89114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenekbool EdgeBuilder::containsLocation(const PathDiagnosticLocation &Container,
89214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek                                   const PathDiagnosticLocation &Containee) {
89314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
89414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  if (Container == Containee)
89514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    return true;
8961eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
89714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  if (Container.asDecl())
89814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    return true;
8991eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
90014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  if (const Stmt *S = Containee.asStmt())
90114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    if (const Stmt *ContainerS = Container.asStmt()) {
90214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      while (S) {
90314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek        if (S == ContainerS)
90414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek          return true;
90514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek        S = PDB.getParent(S);
90614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      }
90714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      return false;
90814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    }
90914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
91014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  // Less accurate: compare using source ranges.
91114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  SourceRange ContainerR = Container.asRange();
91214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  SourceRange ContaineeR = Containee.asRange();
9131eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
91414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  SourceManager &SM = PDB.getSourceManager();
915402785357ab053dd53f4fdd858b9630a5e0f8badChandler Carruth  SourceLocation ContainerRBeg = SM.getExpansionLoc(ContainerR.getBegin());
916402785357ab053dd53f4fdd858b9630a5e0f8badChandler Carruth  SourceLocation ContainerREnd = SM.getExpansionLoc(ContainerR.getEnd());
917402785357ab053dd53f4fdd858b9630a5e0f8badChandler Carruth  SourceLocation ContaineeRBeg = SM.getExpansionLoc(ContaineeR.getBegin());
918402785357ab053dd53f4fdd858b9630a5e0f8badChandler Carruth  SourceLocation ContaineeREnd = SM.getExpansionLoc(ContaineeR.getEnd());
9191eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
920642116259e8df6286063a17361c20e95b5017a0aChandler Carruth  unsigned ContainerBegLine = SM.getExpansionLineNumber(ContainerRBeg);
921642116259e8df6286063a17361c20e95b5017a0aChandler Carruth  unsigned ContainerEndLine = SM.getExpansionLineNumber(ContainerREnd);
922642116259e8df6286063a17361c20e95b5017a0aChandler Carruth  unsigned ContaineeBegLine = SM.getExpansionLineNumber(ContaineeRBeg);
923642116259e8df6286063a17361c20e95b5017a0aChandler Carruth  unsigned ContaineeEndLine = SM.getExpansionLineNumber(ContaineeREnd);
9241eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
92514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  assert(ContainerBegLine <= ContainerEndLine);
9261eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  assert(ContaineeBegLine <= ContaineeEndLine);
9271eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
92814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  return (ContainerBegLine <= ContaineeBegLine &&
92914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek          ContainerEndLine >= ContaineeEndLine &&
93014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek          (ContainerBegLine != ContaineeBegLine ||
931a77c031cb66f75d22672070052cc6e0205289ff8Chandler Carruth           SM.getExpansionColumnNumber(ContainerRBeg) <=
932a77c031cb66f75d22672070052cc6e0205289ff8Chandler Carruth           SM.getExpansionColumnNumber(ContaineeRBeg)) &&
93314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek          (ContainerEndLine != ContaineeEndLine ||
934a77c031cb66f75d22672070052cc6e0205289ff8Chandler Carruth           SM.getExpansionColumnNumber(ContainerREnd) >=
9356488dc31153be6f98b404c7860be6c66bb4ec917Ted Kremenek           SM.getExpansionColumnNumber(ContaineeREnd)));
93614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek}
93714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
93814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenekvoid EdgeBuilder::rawAddEdge(PathDiagnosticLocation NewLoc) {
93914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  if (!PrevLoc.isValid()) {
94014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    PrevLoc = NewLoc;
94114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    return;
94214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  }
9431eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
9448c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek  const PathDiagnosticLocation &NewLocClean = cleanUpLocation(NewLoc);
9458c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek  const PathDiagnosticLocation &PrevLocClean = cleanUpLocation(PrevLoc);
9461eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
9478c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek  if (NewLocClean.asLocation() == PrevLocClean.asLocation())
94814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    return;
9491eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
95014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  // FIXME: Ignore intra-macro edges for now.
951402785357ab053dd53f4fdd858b9630a5e0f8badChandler Carruth  if (NewLocClean.asLocation().getExpansionLoc() ==
952402785357ab053dd53f4fdd858b9630a5e0f8badChandler Carruth      PrevLocClean.asLocation().getExpansionLoc())
95314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    return;
95414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
9552042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek  PD.getActivePath().push_front(new PathDiagnosticControlFlowPiece(NewLocClean, PrevLocClean));
9568c8b0ad9601d6ccf3d7b2a3f77a896ef4fb4e6e9Ted Kremenek  PrevLoc = NewLoc;
95714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek}
95814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
95914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenekvoid EdgeBuilder::addEdge(PathDiagnosticLocation NewLoc, bool alwaysAdd) {
9601eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
961a301a6773db085575ac51e3c966858180390c25bTed Kremenek  if (!alwaysAdd && NewLoc.asLocation().isMacroID())
962a301a6773db085575ac51e3c966858180390c25bTed Kremenek    return;
9631eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
96414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  const PathDiagnosticLocation &CLoc = getContextLocation(NewLoc);
96514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
96614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  while (!CLocs.empty()) {
9678f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek    ContextLocation &TopContextLoc = CLocs.back();
9681eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
96914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    // Is the top location context the same as the one for the new location?
97014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    if (TopContextLoc == CLoc) {
9718f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek      if (alwaysAdd) {
9724c6f8d38e2795c67048e16e737b8225eeb184262Ted Kremenek        if (IsConsumedExpr(TopContextLoc) &&
9734c6f8d38e2795c67048e16e737b8225eeb184262Ted Kremenek            !IsControlFlowExpr(TopContextLoc.asStmt()))
9748f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek            TopContextLoc.markDead();
9758f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek
97614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek        rawAddEdge(NewLoc);
9778f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek      }
97814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
97914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      return;
98014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    }
98114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
98214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    if (containsLocation(TopContextLoc, CLoc)) {
9838f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek      if (alwaysAdd) {
98414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek        rawAddEdge(NewLoc);
9851eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
9864c6f8d38e2795c67048e16e737b8225eeb184262Ted Kremenek        if (IsConsumedExpr(CLoc) && !IsControlFlowExpr(CLoc.asStmt())) {
9878f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek          CLocs.push_back(ContextLocation(CLoc, true));
9888f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek          return;
9898f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek        }
9908f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek      }
9911eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
99214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      CLocs.push_back(CLoc);
9931eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump      return;
99414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    }
99514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
99614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    // Context does not contain the location.  Flush it.
99714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    popLocation();
99814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  }
9991eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
10005c7168cbe8a0eabd05e0e12e469090d2cabd27c0Ted Kremenek  // If we reach here, there is no enclosing context.  Just add the edge.
10015c7168cbe8a0eabd05e0e12e469090d2cabd27c0Ted Kremenek  rawAddEdge(NewLoc);
100214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek}
100314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
10048f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenekbool EdgeBuilder::IsConsumedExpr(const PathDiagnosticLocation &L) {
10058f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek  if (const Expr *X = dyn_cast_or_null<Expr>(L.asStmt()))
10068f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek    return PDB.getParentMap().isConsumedExpr(X) && !IsControlFlowExpr(X);
10071eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
10088f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek  return false;
10098f9b1b3865cd5814a8c4c768a34d56df6d6c93beTed Kremenek}
10101eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1011e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenekvoid EdgeBuilder::addExtendedContext(const Stmt *S) {
1012e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek  if (!S)
1013e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek    return;
10141eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
10151eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  const Stmt *Parent = PDB.getParent(S);
1016e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek  while (Parent) {
1017e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek    if (isa<CompoundStmt>(Parent))
1018e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek      Parent = PDB.getParent(Parent);
1019e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek    else
1020e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek      break;
1021e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek  }
1022e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek
1023e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek  if (Parent) {
1024e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek    switch (Parent->getStmtClass()) {
1025e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek      case Stmt::DoStmtClass:
1026e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek      case Stmt::ObjCAtSynchronizedStmtClass:
1027e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek        addContext(Parent);
1028e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek      default:
1029e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek        break;
1030e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek    }
1031e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek  }
10321eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1033e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek  addContext(S);
1034e1baed38437e08fee7ab372ba2579ce22da354c2Ted Kremenek}
10351eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
103614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenekvoid EdgeBuilder::addContext(const Stmt *S) {
103714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  if (!S)
103814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    return;
103914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
104059950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek  PathDiagnosticLocation L(S, PDB.getSourceManager(), PDB.LC);
1041183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose  addContext(L);
1042183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose}
10431eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1044183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rosevoid EdgeBuilder::addContext(const PathDiagnosticLocation &L) {
104514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  while (!CLocs.empty()) {
104614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    const PathDiagnosticLocation &TopContextLoc = CLocs.back();
104714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
104814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    // Is the top location context the same as the one for the new location?
104914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    if (TopContextLoc == L)
105014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      return;
105114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
105214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    if (containsLocation(TopContextLoc, L)) {
105314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      CLocs.push_back(L);
10541eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump      return;
105514856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    }
105614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
105714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    // Context does not contain the location.  Flush it.
105814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    popLocation();
105914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  }
106014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
106114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  CLocs.push_back(L);
106214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek}
106314856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
106411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// Cone-of-influence: support the reverse propagation of "interesting" symbols
106511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// and values by tracing interesting calculations backwards through evaluated
106611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// expressions along a path.  This is probably overly complicated, but the idea
106711abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// is that if an expression computed an "interesting" value, the child
106811abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// expressions are are also likely to be "interesting" as well (which then
106911abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// propagates to the values they in turn compute).  This reverse propagation
107011abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// is needed to track interesting correlations across function call boundaries,
107111abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// where formal arguments bind to actual arguments, etc.  This is also needed
107211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// because the constraint solver sometimes simplifies certain symbolic values
107311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// into constants when appropriate, and this complicates reasoning about
107411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek// interesting values.
107511abcecc8c919673237cf37384290a1ef1943976Ted Kremenektypedef llvm::DenseSet<const Expr *> InterestingExprs;
107611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek
107711abcecc8c919673237cf37384290a1ef1943976Ted Kremenekstatic void reversePropagateIntererstingSymbols(BugReport &R,
107811abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                                InterestingExprs &IE,
107911abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                                const ProgramState *State,
108011abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                                const Expr *Ex,
108111abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                                const LocationContext *LCtx) {
108211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  SVal V = State->getSVal(Ex, LCtx);
108311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  if (!(R.isInteresting(V) || IE.count(Ex)))
108411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek    return;
108511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek
108611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  switch (Ex->getStmtClass()) {
108711abcecc8c919673237cf37384290a1ef1943976Ted Kremenek    default:
108811abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      if (!isa<CastExpr>(Ex))
108911abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        break;
109011abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      // Fall through.
109111abcecc8c919673237cf37384290a1ef1943976Ted Kremenek    case Stmt::BinaryOperatorClass:
109211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek    case Stmt::UnaryOperatorClass: {
109311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      for (Stmt::const_child_iterator CI = Ex->child_begin(),
109411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek            CE = Ex->child_end();
109511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek            CI != CE; ++CI) {
109611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        if (const Expr *child = dyn_cast_or_null<Expr>(*CI)) {
109711abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          IE.insert(child);
109811abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          SVal ChildV = State->getSVal(child, LCtx);
109911abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          R.markInteresting(ChildV);
110011abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        }
110111abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        break;
110211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      }
110311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek    }
110411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  }
110511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek
110611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  R.markInteresting(V);
110711abcecc8c919673237cf37384290a1ef1943976Ted Kremenek}
110811abcecc8c919673237cf37384290a1ef1943976Ted Kremenek
110911abcecc8c919673237cf37384290a1ef1943976Ted Kremenekstatic void reversePropagateInterestingSymbols(BugReport &R,
111011abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                               InterestingExprs &IE,
111111abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                               const ProgramState *State,
111211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                               const LocationContext *CalleeCtx,
111311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                               const LocationContext *CallerCtx)
111411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek{
1115852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose  // FIXME: Handle non-CallExpr-based CallEvents.
111611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  const StackFrameContext *Callee = CalleeCtx->getCurrentStackFrame();
111711abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  const Stmt *CallSite = Callee->getCallSite();
1118852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose  if (const CallExpr *CE = dyn_cast_or_null<CallExpr>(CallSite)) {
111911abcecc8c919673237cf37384290a1ef1943976Ted Kremenek    if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(CalleeCtx->getDecl())) {
112011abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      FunctionDecl::param_const_iterator PI = FD->param_begin(),
112111abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                         PE = FD->param_end();
112211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      CallExpr::const_arg_iterator AI = CE->arg_begin(), AE = CE->arg_end();
112311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      for (; AI != AE && PI != PE; ++AI, ++PI) {
112411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        if (const Expr *ArgE = *AI) {
112511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          if (const ParmVarDecl *PD = *PI) {
112611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek            Loc LV = State->getLValue(PD, CalleeCtx);
112711abcecc8c919673237cf37384290a1ef1943976Ted Kremenek            if (R.isInteresting(LV) || R.isInteresting(State->getRawSVal(LV)))
112811abcecc8c919673237cf37384290a1ef1943976Ted Kremenek              IE.insert(ArgE);
112911abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          }
113011abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        }
113111abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      }
113211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek    }
113311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  }
113411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek}
113511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek
113614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenekstatic void GenerateExtensivePathDiagnostic(PathDiagnostic& PD,
113714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek                                            PathDiagnosticBuilder &PDB,
11383bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose                                            const ExplodedNode *N,
11393bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose                                      ArrayRef<BugReporterVisitor *> visitors) {
114014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  EdgeBuilder EB(PD, PDB);
11410cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks  const SourceManager& SM = PDB.getSourceManager();
114256a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks  StackDiagVector CallStack;
114311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek  InterestingExprs IE;
114414856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
11459c378f705405d37f49795d5e915989de774fe11fTed Kremenek  const ExplodedNode *NextNode = N->pred_empty() ? NULL : *(N->pred_begin());
114614856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  while (NextNode) {
114714856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    N = NextNode;
114814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    NextNode = GetPredecessorNode(N);
114914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek    ProgramPoint P = N->getLocation();
115014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
1151dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek    do {
115211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      if (const PostStmt *PS = dyn_cast<PostStmt>(&P)) {
115311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        if (const Expr *Ex = PS->getStmtAs<Expr>())
115411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          reversePropagateIntererstingSymbols(*PDB.getBugReport(), IE,
115511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                              N->getState().getPtr(), Ex,
115611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                              N->getLocationContext());
115711abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      }
115811abcecc8c919673237cf37384290a1ef1943976Ted Kremenek
11590b3ade86a1c60cf0c7b56aa238aff458eb7f5974Anna Zaks      if (const CallExitEnd *CE = dyn_cast<CallExitEnd>(&P)) {
1160183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        const Stmt *S = CE->getCalleeContext()->getCallSite();
1161183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        if (const Expr *Ex = dyn_cast_or_null<Expr>(S)) {
1162852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose            reversePropagateIntererstingSymbols(*PDB.getBugReport(), IE,
1163852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose                                                N->getState().getPtr(), Ex,
1164852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose                                                N->getLocationContext());
1165852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose        }
1166183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose
1167183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        PathDiagnosticCallPiece *C =
1168183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose          PathDiagnosticCallPiece::construct(N, *CE, SM);
1169183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose
1170183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        EB.addEdge(C->callReturn, true);
1171183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        EB.flushLocations();
1172183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose
1173183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        PD.getActivePath().push_front(C);
1174183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        PD.pushActivePath(&C->path);
1175183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        CallStack.push_back(StackDiagPair(C, N));
11765de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek        break;
11775de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek      }
11784ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek
11792042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      // Pop the call hierarchy if we are done walking the contents
11802042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      // of a function call.
11812042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      if (const CallEnter *CE = dyn_cast<CallEnter>(&P)) {
1182097ebb3d8ce55d1f78a3f1e7a0978dbde5ee2898Ted Kremenek        // Add an edge to the start of the function.
1183097ebb3d8ce55d1f78a3f1e7a0978dbde5ee2898Ted Kremenek        const Decl *D = CE->getCalleeContext()->getDecl();
1184097ebb3d8ce55d1f78a3f1e7a0978dbde5ee2898Ted Kremenek        PathDiagnosticLocation pos =
1185097ebb3d8ce55d1f78a3f1e7a0978dbde5ee2898Ted Kremenek          PathDiagnosticLocation::createBegin(D, SM);
1186097ebb3d8ce55d1f78a3f1e7a0978dbde5ee2898Ted Kremenek        EB.addEdge(pos);
1187097ebb3d8ce55d1f78a3f1e7a0978dbde5ee2898Ted Kremenek
1188097ebb3d8ce55d1f78a3f1e7a0978dbde5ee2898Ted Kremenek        // Flush all locations, and pop the active path.
1189183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        bool VisitedEntireCall = PD.isWithinCall();
11904ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek        EB.flushLocations();
11912042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek        PD.popActivePath();
11924ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek        PDB.LC = N->getLocationContext();
1193097ebb3d8ce55d1f78a3f1e7a0978dbde5ee2898Ted Kremenek
1194183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        // Either we just added a bunch of stuff to the top-level path, or
1195183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        // we have a previous CallExitEnd.  If the former, it means that the
11962042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek        // path terminated within a function call.  We must then take the
11972042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek        // current contents of the active path and place it within
11982042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek        // a new PathDiagnosticCallPiece.
1199183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        PathDiagnosticCallPiece *C;
1200183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        if (VisitedEntireCall) {
1201183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose          C = cast<PathDiagnosticCallPiece>(PD.getActivePath().front());
1202183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        } else {
1203183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose          const Decl *Caller = CE->getLocationContext()->getDecl();
12049373937945e1e075dfa08169eaccc1ad0b31f699Anna Zaks          C = PathDiagnosticCallPiece::construct(PD.getActivePath(), Caller);
12059373937945e1e075dfa08169eaccc1ad0b31f699Anna Zaks        }
1206852aa0d2c5d2d1faf2d77b5aa3c0848068a342c5Jordan Rose
1207183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        C->setCallee(*CE, SM);
1208183ba8e19d49ab1ae25d3cdd0a19591369c5ab9fJordan Rose        EB.addContext(C->getLocation());
1209368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks
1210368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        if (!CallStack.empty()) {
121156a938ff85a444eb3d30d2634d92ce5b1f6fae56Anna Zaks          assert(CallStack.back().first == C);
1212368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks          CallStack.pop_back();
1213368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        }
12142042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek        break;
12152042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek      }
12164ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek
12174ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek      // Note that is important that we update the LocationContext
12184ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek      // after looking at CallExits.  CallExit basically adds an
12194ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek      // edge in the *caller*, so we don't want to update the LocationContext
12204ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek      // too soon.
12214ba86bc53bb280ba46a08459eda7d283d513b61fTed Kremenek      PDB.LC = N->getLocationContext();
12225de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek
1223dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek      // Block edges.
122411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek      if (const BlockEdge *BE = dyn_cast<BlockEdge>(&P)) {
122511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        // Does this represent entering a call?  If so, look at propagating
122611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        // interesting symbols across call boundaries.
122711abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        if (NextNode) {
122811abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          const LocationContext *CallerCtx = NextNode->getLocationContext();
122911abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          const LocationContext *CalleeCtx = PDB.LC;
123011abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          if (CallerCtx != CalleeCtx) {
123111abcecc8c919673237cf37384290a1ef1943976Ted Kremenek            reversePropagateInterestingSymbols(*PDB.getBugReport(), IE,
123211abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                               N->getState().getPtr(),
123311abcecc8c919673237cf37384290a1ef1943976Ted Kremenek                                               CalleeCtx, CallerCtx);
123411abcecc8c919673237cf37384290a1ef1943976Ted Kremenek          }
123511abcecc8c919673237cf37384290a1ef1943976Ted Kremenek        }
123611abcecc8c919673237cf37384290a1ef1943976Ted Kremenek
1237dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek        const CFGBlock &Blk = *BE->getSrc();
1238dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek        const Stmt *Term = Blk.getTerminator();
12391eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1240dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek        // Are we jumping to the head of a loop?  Add a special diagnostic.
1241ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek        if (const Stmt *Loop = BE->getDst()->getLoopTarget()) {
124259950d3aa54ca5066b1fb08a8c79ebfe10e0919bTed Kremenek          PathDiagnosticLocation L(Loop, SM, PDB.LC);
1243ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek          const CompoundStmt *CS = NULL;
12441eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1245ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek          if (!Term) {
1246ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek            if (const ForStmt *FS = dyn_cast<ForStmt>(Loop))
1247ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek              CS = dyn_cast<CompoundStmt>(FS->getBody());
1248ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek            else if (const WhileStmt *WS = dyn_cast<WhileStmt>(Loop))
12491eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump              CS = dyn_cast<CompoundStmt>(WS->getBody());
1250ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek          }
12511eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1252dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek          PathDiagnosticEventPiece *p =
1253dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek            new PathDiagnosticEventPiece(L,
125407c015cbcee31f72e2d320ba2c713c046bed42faTed Kremenek                                        "Looping back to the head of the loop");
12552dd17abf11ae64339fa6bfaa57d76e13a5fbe5b8Ted Kremenek          p->setPrunable(true);
12561eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1257dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek          EB.addEdge(p->getLocation(), true);
12582042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek          PD.getActivePath().push_front(p);
12591eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1260ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek          if (CS) {
12610cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks            PathDiagnosticLocation BL =
12620cd59482abd8aec9ed1eaad11f5fe9c1e42639f6Anna Zaks              PathDiagnosticLocation::createEndBrace(CS, SM);
126307c015cbcee31f72e2d320ba2c713c046bed42faTed Kremenek            EB.addEdge(BL);
1264dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek          }
12658bd4d037d837e7922c3661d6158229da58a03887Ted Kremenek        }
12661eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1267ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek        if (Term)
1268ddb7babf448822e0f8da07fdf9446cf515d04ad5Ted Kremenek          EB.addContext(Term);
12691eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1270dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek        break;
12718bd4d037d837e7922c3661d6158229da58a03887Ted Kremenek      }
127214856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
12731eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump      if (const BlockEntrance *BE = dyn_cast<BlockEntrance>(&P)) {
12743c0349e87cdbd7316d06d2411d86ee1086e717a5Ted Kremenek        if (const CFGStmt *S = BE->getFirstElement().getAs<CFGStmt>()) {
12753c0349e87cdbd7316d06d2411d86ee1086e717a5Ted Kremenek          const Stmt *stmt = S->getStmt();
12763c0349e87cdbd7316d06d2411d86ee1086e717a5Ted Kremenek          if (IsControlFlowExpr(stmt)) {
1277b36cd3e1757fb4fcd9509f35558c847b04bef35fZhongxing Xu            // Add the proper context for '&&', '||', and '?'.
12783c0349e87cdbd7316d06d2411d86ee1086e717a5Ted Kremenek            EB.addContext(stmt);
1279b36cd3e1757fb4fcd9509f35558c847b04bef35fZhongxing Xu          }
1280b36cd3e1757fb4fcd9509f35558c847b04bef35fZhongxing Xu          else
12813c0349e87cdbd7316d06d2411d86ee1086e717a5Ted Kremenek            EB.addExtendedContext(PDB.getEnclosingStmtLocation(stmt).asStmt());
1282dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek        }
1283b36cd3e1757fb4fcd9509f35558c847b04bef35fZhongxing Xu
1284dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek        break;
1285dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek      }
12865de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek
12875de4fdb8de700f95b0b863a9e5a4a508de17a034Ted Kremenek
1288dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek    } while (0);
12891eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1290dd986cc9989f665370cef0917ba8ba3b4871e3e6Ted Kremenek    if (!NextNode)
129114856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek      continue;
12921eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
12938e6431adab313e283a992698f6fc7afe62420999Anna Zaks    // Add pieces from custom visitors.
12948e6431adab313e283a992698f6fc7afe62420999Anna Zaks    BugReport *R = PDB.getBugReport();
12953bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    for (ArrayRef<BugReporterVisitor *>::iterator I = visitors.begin(),
12963bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose                                                  E = visitors.end();
12973bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose         I != E; ++I) {
12988e6431adab313e283a992698f6fc7afe62420999Anna Zaks      if (PathDiagnosticPiece *p = (*I)->VisitNode(N, NextNode, PDB, *R)) {
12998966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek        const PathDiagnosticLocation &Loc = p->getLocation();
13008966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek        EB.addEdge(Loc, true);
13012042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek        PD.getActivePath().push_front(p);
1302368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks        updateStackPiecesWithMessage(p, CallStack);
1303368a0d565f078666ca5bfb7fe08d04648688e4bcAnna Zaks
13048966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek        if (const Stmt *S = Loc.asStmt())
13051eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump          EB.addExtendedContext(PDB.getEnclosingStmtLocation(S).asStmt());
13068966bc1c8ce271c09936c0eaf6c841aef4a0af1bTed Kremenek      }
13071eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump    }
130814856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek  }
130914856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek}
131014856d7b7a2be05ae9a0fd5b2073631994c2ba43Ted Kremenek
13115fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek//===----------------------------------------------------------------------===//
13123106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek// Methods for BugType and subclasses.
13133106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek//===----------------------------------------------------------------------===//
1314404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios KyrtzidisBugType::~BugType() { }
1315404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis
13163106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenekvoid BugType::FlushReports(BugReporter &BR) {}
13173106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
131899ba9e3bd70671f3441fb974895f226a83ce0e66David Blaikievoid BuiltinBug::anchor() {}
131999ba9e3bd70671f3441fb974895f226a83ce0e66David Blaikie
13203106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek//===----------------------------------------------------------------------===//
13213106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek// Methods for BugReport and subclasses.
13223106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek//===----------------------------------------------------------------------===//
1323e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks
132499ba9e3bd70671f3441fb974895f226a83ce0e66David Blaikievoid BugReport::NodeResolver::anchor() {}
132599ba9e3bd70671f3441fb974895f226a83ce0e66David Blaikie
13268e6431adab313e283a992698f6fc7afe62420999Anna Zaksvoid BugReport::addVisitor(BugReporterVisitor* visitor) {
13278e6431adab313e283a992698f6fc7afe62420999Anna Zaks  if (!visitor)
13288e6431adab313e283a992698f6fc7afe62420999Anna Zaks    return;
13298e6431adab313e283a992698f6fc7afe62420999Anna Zaks
13308e6431adab313e283a992698f6fc7afe62420999Anna Zaks  llvm::FoldingSetNodeID ID;
13318e6431adab313e283a992698f6fc7afe62420999Anna Zaks  visitor->Profile(ID);
13328e6431adab313e283a992698f6fc7afe62420999Anna Zaks  void *InsertPos;
13338e6431adab313e283a992698f6fc7afe62420999Anna Zaks
13348e6431adab313e283a992698f6fc7afe62420999Anna Zaks  if (CallbacksSet.FindNodeOrInsertPos(ID, InsertPos)) {
13358e6431adab313e283a992698f6fc7afe62420999Anna Zaks    delete visitor;
13368e6431adab313e283a992698f6fc7afe62420999Anna Zaks    return;
13378e6431adab313e283a992698f6fc7afe62420999Anna Zaks  }
13388e6431adab313e283a992698f6fc7afe62420999Anna Zaks
13398e6431adab313e283a992698f6fc7afe62420999Anna Zaks  CallbacksSet.InsertNode(visitor, InsertPos);
13403bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  Callbacks.push_back(visitor);
13413bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  ++ConfigurationChangeToken;
13428e6431adab313e283a992698f6fc7afe62420999Anna Zaks}
13438e6431adab313e283a992698f6fc7afe62420999Anna Zaks
13448e6431adab313e283a992698f6fc7afe62420999Anna ZaksBugReport::~BugReport() {
13458e6431adab313e283a992698f6fc7afe62420999Anna Zaks  for (visitor_iterator I = visitor_begin(), E = visitor_end(); I != E; ++I) {
1346dc757b049796949e4b11646445a6598f0bdabd7aAnna Zaks    delete *I;
13478e6431adab313e283a992698f6fc7afe62420999Anna Zaks  }
13488e6431adab313e283a992698f6fc7afe62420999Anna Zaks}
1349e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks
135007189521a15d9c088216b943649cb9fe231cbb57Ted Kremenekconst Decl *BugReport::getDeclWithIssue() const {
135107189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek  if (DeclWithIssue)
135207189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek    return DeclWithIssue;
135307189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek
135407189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek  const ExplodedNode *N = getErrorNode();
135507189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek  if (!N)
135607189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek    return 0;
135707189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek
135807189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek  const LocationContext *LC = N->getLocationContext();
135907189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek  return LC->getCurrentStackFrame()->getDecl();
136007189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek}
136107189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek
1362e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaksvoid BugReport::Profile(llvm::FoldingSetNodeID& hash) const {
1363e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks  hash.AddPointer(&BT);
1364e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks  hash.AddString(Description);
1365ca8e36eb637e232475ef31c3f22d5da907390917Anna Zaks  if (UniqueingLocation.isValid()) {
1366ca8e36eb637e232475ef31c3f22d5da907390917Anna Zaks    UniqueingLocation.Profile(hash);
1367ca8e36eb637e232475ef31c3f22d5da907390917Anna Zaks  } else if (Location.isValid()) {
1368590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks    Location.Profile(hash);
1369590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks  } else {
1370590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks    assert(ErrorNode);
1371590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks    hash.AddPointer(GetCurrentOrPreviousStmt(ErrorNode));
1372590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks  }
1373e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks
1374e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks  for (SmallVectorImpl<SourceRange>::const_iterator I =
1375e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks      Ranges.begin(), E = Ranges.end(); I != E; ++I) {
1376e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    const SourceRange range = *I;
1377e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    if (!range.isValid())
1378e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks      continue;
1379e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    hash.AddInteger(range.getBegin().getRawEncoding());
1380e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    hash.AddInteger(range.getEnd().getRawEncoding());
1381e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks  }
1382e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks}
13833106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
138476aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenekvoid BugReport::markInteresting(SymbolRef sym) {
138576aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  if (!sym)
138676aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek    return;
13873bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose
13883bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  // If the symbol wasn't already in our set, note a configuration change.
13893bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  if (interestingSymbols.insert(sym).second)
13903bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    ++ConfigurationChangeToken;
13918ec588e2ac57311604cf80608c7d4b3fb3b022f7Jordy Rose
13928ec588e2ac57311604cf80608c7d4b3fb3b022f7Jordy Rose  if (const SymbolMetadata *meta = dyn_cast<SymbolMetadata>(sym))
13938ec588e2ac57311604cf80608c7d4b3fb3b022f7Jordy Rose    interestingRegions.insert(meta->getRegion());
139476aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek}
139576aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek
139676aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenekvoid BugReport::markInteresting(const MemRegion *R) {
139776aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  if (!R)
139876aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek    return;
13993bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose
14003bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  // If the base region wasn't already in our set, note a configuration change.
140176aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  R = R->getBaseRegion();
14023bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  if (interestingRegions.insert(R).second)
14033bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    ++ConfigurationChangeToken;
14048ec588e2ac57311604cf80608c7d4b3fb3b022f7Jordy Rose
140576aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  if (const SymbolicRegion *SR = dyn_cast<SymbolicRegion>(R))
140676aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek    interestingSymbols.insert(SR->getSymbol());
140776aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek}
140876aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek
140976aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenekvoid BugReport::markInteresting(SVal V) {
141076aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  markInteresting(V.getAsRegion());
141176aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  markInteresting(V.getAsSymbol());
141276aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek}
141376aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek
141476aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenekbool BugReport::isInteresting(SVal V) const {
141576aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  return isInteresting(V.getAsRegion()) || isInteresting(V.getAsSymbol());
141676aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek}
141776aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek
141876aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenekbool BugReport::isInteresting(SymbolRef sym) const {
141976aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  if (!sym)
142076aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek    return false;
14218ec588e2ac57311604cf80608c7d4b3fb3b022f7Jordy Rose  // We don't currently consider metadata symbols to be interesting
14228ec588e2ac57311604cf80608c7d4b3fb3b022f7Jordy Rose  // even if we know their region is interesting. Is that correct behavior?
142376aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  return interestingSymbols.count(sym);
142476aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek}
142576aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek
142676aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenekbool BugReport::isInteresting(const MemRegion *R) const {
142776aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  if (!R)
142876aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek    return false;
142976aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  R = R->getBaseRegion();
143076aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  bool b = interestingRegions.count(R);
143176aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  if (b)
143276aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek    return true;
143376aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  if (const SymbolicRegion *SR = dyn_cast<SymbolicRegion>(R))
143476aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek    return interestingSymbols.count(SR->getSymbol());
143576aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek  return false;
143676aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek}
143776aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek
143876aadc346c3a4c363238a1e1232f324c3355d9e0Ted Kremenek
14399c378f705405d37f49795d5e915989de774fe11fTed Kremenekconst Stmt *BugReport::getStmt() const {
1440e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks  if (!ErrorNode)
1441e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    return 0;
1442e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks
1443212f6d3b5fb3fa55ba1e40671cfc336430abc8ddTom Care  ProgramPoint ProgP = ErrorNode->getLocation();
14445f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenek  const Stmt *S = NULL;
14451eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
14469c378f705405d37f49795d5e915989de774fe11fTed Kremenek  if (BlockEntrance *BE = dyn_cast<BlockEntrance>(&ProgP)) {
1447fafd3834754d2093e0ad7a1c005860fd527ecb7fZhongxing Xu    CFGBlock &Exit = ProgP.getLocationContext()->getCFG()->getExit();
144850d5bc4e3b53eb8bb4ebf198f1213348a3fa0f38Zhongxing Xu    if (BE->getBlock() == &Exit)
1449212f6d3b5fb3fa55ba1e40671cfc336430abc8ddTom Care      S = GetPreviousStmt(ErrorNode);
14503106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  }
14515f85e17df3f5b0a8021443f2b590daecfb2cbd17Ted Kremenek  if (!S)
14521eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump    S = GetStmt(ProgP);
14531eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
14541eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  return S;
14553106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek}
14563106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
1457640ccf071076e684713cc3c3276bb51982bff607Argyrios Kyrtzidisstd::pair<BugReport::ranges_iterator, BugReport::ranges_iterator>
1458e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna ZaksBugReport::getRanges() {
1459e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    // If no custom ranges, add the range of the statement corresponding to
1460e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    // the error node.
1461e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    if (Ranges.empty()) {
1462e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks      if (const Expr *E = dyn_cast_or_null<Expr>(getStmt()))
1463e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks        addRange(E->getSourceRange());
1464e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks      else
1465e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks        return std::make_pair(ranges_iterator(), ranges_iterator());
1466e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    }
1467e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks
146814924267950e75c6c1f6fcea39fa507b7168bc39Anna Zaks    // User-specified absence of range info.
146914924267950e75c6c1f6fcea39fa507b7168bc39Anna Zaks    if (Ranges.size() == 1 && !Ranges.begin()->isValid())
147014924267950e75c6c1f6fcea39fa507b7168bc39Anna Zaks      return std::make_pair(ranges_iterator(), ranges_iterator());
147114924267950e75c6c1f6fcea39fa507b7168bc39Anna Zaks
1472e172e8b9e7fc67d7d03589af7e92fe777afcf33aAnna Zaks    return std::make_pair(Ranges.begin(), Ranges.end());
14733106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek}
14743106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
1475590dd8e0959d8df5621827768987c4792b74fc06Anna ZaksPathDiagnosticLocation BugReport::getLocation(const SourceManager &SM) const {
1476b7530a4ca9a7ef62350682bbb374a06de6fdaa9fAnna Zaks  if (ErrorNode) {
1477590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks    assert(!Location.isValid() &&
1478b7530a4ca9a7ef62350682bbb374a06de6fdaa9fAnna Zaks     "Either Location or ErrorNode should be specified but not both.");
1479b7530a4ca9a7ef62350682bbb374a06de6fdaa9fAnna Zaks
14809c378f705405d37f49795d5e915989de774fe11fTed Kremenek    if (const Stmt *S = GetCurrentOrPreviousStmt(ErrorNode)) {
1481590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks      const LocationContext *LC = ErrorNode->getLocationContext();
1482590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks
14833106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      // For member expressions, return the location of the '.' or '->'.
14845b9bd2137ebef350af803c634e3fdf5d74678100Ted Kremenek      if (const MemberExpr *ME = dyn_cast<MemberExpr>(S))
1485590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks        return PathDiagnosticLocation::createMemberLoc(ME, SM);
14865b9bd2137ebef350af803c634e3fdf5d74678100Ted Kremenek      // For binary operators, return the location of the operator.
14875b9bd2137ebef350af803c634e3fdf5d74678100Ted Kremenek      if (const BinaryOperator *B = dyn_cast<BinaryOperator>(S))
1488590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks        return PathDiagnosticLocation::createOperatorLoc(B, SM);
14893106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
1490590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks      return PathDiagnosticLocation::createBegin(S, SM, LC);
14913106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    }
1492b7530a4ca9a7ef62350682bbb374a06de6fdaa9fAnna Zaks  } else {
1493b7530a4ca9a7ef62350682bbb374a06de6fdaa9fAnna Zaks    assert(Location.isValid());
1494b7530a4ca9a7ef62350682bbb374a06de6fdaa9fAnna Zaks    return Location;
1495b7530a4ca9a7ef62350682bbb374a06de6fdaa9fAnna Zaks  }
1496b7530a4ca9a7ef62350682bbb374a06de6fdaa9fAnna Zaks
1497590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks  return PathDiagnosticLocation();
14983106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek}
14993106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
15003106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek//===----------------------------------------------------------------------===//
15013106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek// Methods for BugReporter and subclasses.
15023106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek//===----------------------------------------------------------------------===//
15033106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
15044a5f724538cbc275370c9504e8169ce92503256cBenjamin KramerBugReportEquivClass::~BugReportEquivClass() { }
1505a2f4ec0df645fc249d2945beef9653f03b175417Zhongxing XuGRBugReporter::~GRBugReporter() { }
15063106198188ac6eb2753f1764d5c28376b0b76351Ted KremenekBugReporterData::~BugReporterData() {}
15073106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
150838b02b912e1a55c912f603c4369431264d36a381Zhongxing XuExplodedGraph &GRBugReporter::getGraph() { return Eng.getGraph(); }
15093106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
151018c66fdc3c4008d335885695fe36fb5353c5f672Ted KremenekProgramStateManager&
15113106198188ac6eb2753f1764d5c28376b0b76351Ted KremenekGRBugReporter::getStateManager() { return Eng.getStateManager(); }
15123106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
15133b030a28cda2b953758507769c1d436bec5ec45eAnna ZaksBugReporter::~BugReporter() {
15143b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks  FlushReports();
15153b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks
15163b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks  // Free the bug reports we are tracking.
15173b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks  typedef std::vector<BugReportEquivClass *> ContTy;
15183b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks  for (ContTy::iterator I = EQClassesVector.begin(), E = EQClassesVector.end();
15193b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks       I != E; ++I) {
15203b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks    delete *I;
15213b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks  }
15223b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks}
15233106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
15243106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenekvoid BugReporter::FlushReports() {
15253106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  if (BugTypes.isEmpty())
15263106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    return;
15273106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
15283106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // First flush the warnings for each BugType.  This may end up creating new
1529404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  // warnings and new BugTypes.
1530404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  // FIXME: Only NSErrorChecker needs BugType's FlushReports.
1531404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  // Turn NSErrorChecker into a proper checker and remove this.
15325f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner  SmallVector<const BugType*, 16> bugTypes;
15333106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  for (BugTypesTy::iterator I=BugTypes.begin(), E=BugTypes.end(); I!=E; ++I)
1534404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis    bugTypes.push_back(*I);
15355f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner  for (SmallVector<const BugType*, 16>::iterator
1536404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis         I = bugTypes.begin(), E = bugTypes.end(); I != E; ++I)
15373106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    const_cast<BugType*>(*I)->FlushReports(*this);
15383106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
15390eb6c37dd4e4ad8fa2363003dea270f9fd6c2969Anna Zaks  typedef std::vector<BugReportEquivClass *> ContVecTy;
15400eb6c37dd4e4ad8fa2363003dea270f9fd6c2969Anna Zaks  for (ContVecTy::iterator EI=EQClassesVector.begin(), EE=EQClassesVector.end();
15410eb6c37dd4e4ad8fa2363003dea270f9fd6c2969Anna Zaks       EI != EE; ++EI){
15420eb6c37dd4e4ad8fa2363003dea270f9fd6c2969Anna Zaks    BugReportEquivClass& EQ = **EI;
1543404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis    FlushReport(EQ);
15443106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  }
15453106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
1546404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  // BugReporter owns and deletes only BugTypes created implicitly through
1547404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  // EmitBasicReport.
1548404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  // FIXME: There are leaks from checkers that assume that the BugTypes they
1549404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  // create will be destroyed by the BugReporter.
1550404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  for (llvm::StringMap<BugType*>::iterator
1551404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis         I = StrBugTypes.begin(), E = StrBugTypes.end(); I != E; ++I)
1552404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis    delete I->second;
1553404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis
15543106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Remove all references to the BugType objects.
15553baf672378f105602d2b12f03f00277ae1936fe9Ted Kremenek  BugTypes = F.getEmptySet();
15563106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek}
15573106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
15583106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek//===----------------------------------------------------------------------===//
15593106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek// PathDiagnostics generation.
15603106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek//===----------------------------------------------------------------------===//
15613106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
156238b02b912e1a55c912f603c4369431264d36a381Zhongxing Xustatic std::pair<std::pair<ExplodedGraph*, NodeBackMap*>,
1563c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu                 std::pair<ExplodedNode*, unsigned> >
156438b02b912e1a55c912f603c4369431264d36a381Zhongxing XuMakeReportGraph(const ExplodedGraph* G,
15655f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner                SmallVectorImpl<const ExplodedNode*> &nodes) {
15661eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
15673106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Create the trimmed graph.  It will contain the shortest paths from the
15681eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  // error nodes to the root.  In the new graph we should only have one
15693106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // error node unless there are two or more error nodes with the same minimum
15703106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // path length.
157138b02b912e1a55c912f603c4369431264d36a381Zhongxing Xu  ExplodedGraph* GTrim;
1572c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu  InterExplodedGraphMap* NMap;
15733106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
15743106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  llvm::DenseMap<const void*, const void*> InverseMap;
157540406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  llvm::tie(GTrim, NMap) = G->Trim(nodes.data(), nodes.data() + nodes.size(),
157640406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek                                   &InverseMap);
15771eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
15783106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Create owning pointers for GTrim and NMap just to ensure that they are
15793106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // released when this function exists.
15806f42b62b6194f53bcbc349f5d17388e1936535d7Dylan Noblesmith  OwningPtr<ExplodedGraph> AutoReleaseGTrim(GTrim);
15816f42b62b6194f53bcbc349f5d17388e1936535d7Dylan Noblesmith  OwningPtr<InterExplodedGraphMap> AutoReleaseNMap(NMap);
15821eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
15833106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Find the (first) error node in the trimmed graph.  We just need to consult
15843106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // the node map (NMap) which maps from nodes in the original graph to nodes
15853106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // in the new graph.
1586938332c657390d1e782e0adc03b092993edae962Ted Kremenek
1587c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu  std::queue<const ExplodedNode*> WS;
158838b02b912e1a55c912f603c4369431264d36a381Zhongxing Xu  typedef llvm::DenseMap<const ExplodedNode*, unsigned> IndexMapTy;
1589938332c657390d1e782e0adc03b092993edae962Ted Kremenek  IndexMapTy IndexMap;
15903106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
159140406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  for (unsigned nodeIndex = 0 ; nodeIndex < nodes.size(); ++nodeIndex) {
159240406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    const ExplodedNode *originalNode = nodes[nodeIndex];
159340406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    if (const ExplodedNode *N = NMap->getMappedNode(originalNode)) {
1594938332c657390d1e782e0adc03b092993edae962Ted Kremenek      WS.push(N);
159540406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek      IndexMap[originalNode] = nodeIndex;
15963106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    }
159740406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  }
15981eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1599938332c657390d1e782e0adc03b092993edae962Ted Kremenek  assert(!WS.empty() && "No error node found in the trimmed graph.");
16003106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
16013106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Create a new (third!) graph with a single path.  This is the graph
16023106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // that will be returned to the caller.
1603c77a55126fcad66fb086f8e100a494caa2496a2dZhongxing Xu  ExplodedGraph *GNew = new ExplodedGraph();
16041eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16053106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Sometimes the trimmed graph can contain a cycle.  Perform a reverse BFS
16063106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // to the root node, and then construct a new graph that contains only
16073106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // a single path.
16083106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  llvm::DenseMap<const void*,unsigned> Visited;
16091eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16103106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  unsigned cnt = 0;
16119c378f705405d37f49795d5e915989de774fe11fTed Kremenek  const ExplodedNode *Root = 0;
16121eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16133106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  while (!WS.empty()) {
16149c378f705405d37f49795d5e915989de774fe11fTed Kremenek    const ExplodedNode *Node = WS.front();
16153106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    WS.pop();
16161eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16173106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    if (Visited.find(Node) != Visited.end())
16183106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      continue;
16191eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16203106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    Visited[Node] = cnt++;
16211eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16223106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    if (Node->pred_empty()) {
16233106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      Root = Node;
16243106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      break;
16253106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    }
16261eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1627c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu    for (ExplodedNode::const_pred_iterator I=Node->pred_begin(),
16283106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek         E=Node->pred_end(); I!=E; ++I)
16293106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      WS.push(*I);
16303106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  }
16311eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1632938332c657390d1e782e0adc03b092993edae962Ted Kremenek  assert(Root);
16331eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16343106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Now walk from the root down the BFS path, always taking the successor
16353106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // with the lowest number.
16361eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  ExplodedNode *Last = 0, *First = 0;
16373106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  NodeBackMap *BM = new NodeBackMap();
1638938332c657390d1e782e0adc03b092993edae962Ted Kremenek  unsigned NodeIndex = 0;
16391eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1640c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu  for ( const ExplodedNode *N = Root ;;) {
16413106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Lookup the number associated with the current node.
16423106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    llvm::DenseMap<const void*,unsigned>::iterator I = Visited.find(N);
1643938332c657390d1e782e0adc03b092993edae962Ted Kremenek    assert(I != Visited.end());
16441eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16453106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Create the equivalent node in the new graph with the same state
16463106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // and location.
16479c378f705405d37f49795d5e915989de774fe11fTed Kremenek    ExplodedNode *NewN = GNew->getNode(N->getLocation(), N->getState());
16481eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16493106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Store the mapping to the original node.
16503106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    llvm::DenseMap<const void*, const void*>::iterator IMitr=InverseMap.find(N);
16513106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    assert(IMitr != InverseMap.end() && "No mapping to original node.");
1652c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu    (*BM)[NewN] = (const ExplodedNode*) IMitr->second;
16531eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16543106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Link up the new node with the previous node.
16553106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    if (Last)
16565fe4d9deb543a19f557e3d85c5f33867af97cd96Ted Kremenek      NewN->addPredecessor(Last, *GNew);
16571eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16583106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    Last = NewN;
16591eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16603106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Are we at the final node?
1661938332c657390d1e782e0adc03b092993edae962Ted Kremenek    IndexMapTy::iterator IMI =
1662c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu      IndexMap.find((const ExplodedNode*)(IMitr->second));
1663938332c657390d1e782e0adc03b092993edae962Ted Kremenek    if (IMI != IndexMap.end()) {
16643106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      First = NewN;
1665938332c657390d1e782e0adc03b092993edae962Ted Kremenek      NodeIndex = IMI->second;
16663106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      break;
16673106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    }
16681eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16693106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Find the next successor node.  We choose the node that is marked
16703106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // with the lowest DFS number.
1671c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu    ExplodedNode::const_succ_iterator SI = N->succ_begin();
1672c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu    ExplodedNode::const_succ_iterator SE = N->succ_end();
16733106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    N = 0;
16741eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16753106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    for (unsigned MinVal = 0; SI != SE; ++SI) {
16761eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16773106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      I = Visited.find(*SI);
16781eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16793106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      if (I == Visited.end())
16803106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        continue;
16811eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
16823106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      if (!N || I->second < MinVal) {
16833106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        N = *SI;
16843106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        MinVal = I->second;
16853106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      }
16863106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    }
16871eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1688938332c657390d1e782e0adc03b092993edae962Ted Kremenek    assert(N);
16893106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  }
16901eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1691938332c657390d1e782e0adc03b092993edae962Ted Kremenek  assert(First);
1692938332c657390d1e782e0adc03b092993edae962Ted Kremenek
16933106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  return std::make_pair(std::make_pair(GNew, BM),
16943106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek                        std::make_pair(First, NodeIndex));
16953106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek}
1696d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek
16973106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek/// CompactPathDiagnostic - This function postprocesses a PathDiagnostic object
16983106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek///  and collapses PathDiagosticPieces that are expanded by macros.
169977d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenekstatic void CompactPathDiagnostic(PathPieces &path, const SourceManager& SM) {
17002042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek  typedef std::vector<std::pair<IntrusiveRefCntPtr<PathDiagnosticMacroPiece>,
17012042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek                                SourceLocation> > MacroStackTy;
17021eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1703c93dc7889644293e318e19d82830ea2acc45b678Dylan Noblesmith  typedef std::vector<IntrusiveRefCntPtr<PathDiagnosticPiece> >
17043106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek          PiecesTy;
17051eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17063106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  MacroStackTy MacroStack;
17073106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  PiecesTy Pieces;
17081eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
170977d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek  for (PathPieces::const_iterator I = path.begin(), E = path.end();
17102042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek       I!=E; ++I) {
171177d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek
171277d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek    PathDiagnosticPiece *piece = I->getPtr();
171377d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek
171477d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek    // Recursively compact calls.
171577d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek    if (PathDiagnosticCallPiece *call=dyn_cast<PathDiagnosticCallPiece>(piece)){
171677d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek      CompactPathDiagnostic(call->path, SM);
171777d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek    }
171877d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek
17193106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Get the location of the PathDiagnosticPiece.
172077d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek    const FullSourceLoc Loc = piece->getLocation().asLocation();
17211eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17223106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Determine the instantiation location, which is the location we group
17233106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // related PathDiagnosticPieces.
17241eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump    SourceLocation InstantiationLoc = Loc.isMacroID() ?
1725402785357ab053dd53f4fdd858b9630a5e0f8badChandler Carruth                                      SM.getExpansionLoc(Loc) :
17263106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek                                      SourceLocation();
17271eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17283106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    if (Loc.isFileID()) {
17293106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      MacroStack.clear();
173077d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek      Pieces.push_back(piece);
17313106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      continue;
17323106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    }
1733706e3cf2fa7523b5497c52c11a2b6dad2a1ce2acTed Kremenek
17343106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    assert(Loc.isMacroID());
17351eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17363106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Is the PathDiagnosticPiece within the same macro group?
17373106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    if (!MacroStack.empty() && InstantiationLoc == MacroStack.back().second) {
173877d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek      MacroStack.back().first->subPieces.push_back(piece);
17393106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      continue;
17403106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    }
1741d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek
17423106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // We aren't in the same group.  Are we descending into a new macro
17433106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // or are part of an old one?
1744c93dc7889644293e318e19d82830ea2acc45b678Dylan Noblesmith    IntrusiveRefCntPtr<PathDiagnosticMacroPiece> MacroGroup;
1745d8c938b0f6b7a7156181be575239e4c6e15a2adbTed Kremenek
17463106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    SourceLocation ParentInstantiationLoc = InstantiationLoc.isMacroID() ?
1747402785357ab053dd53f4fdd858b9630a5e0f8badChandler Carruth                                          SM.getExpansionLoc(Loc) :
17483106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek                                          SourceLocation();
17491eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17503106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Walk the entire macro stack.
17513106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    while (!MacroStack.empty()) {
17523106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      if (InstantiationLoc == MacroStack.back().second) {
17533106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        MacroGroup = MacroStack.back().first;
17543106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        break;
17553106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      }
17561eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17573106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      if (ParentInstantiationLoc == MacroStack.back().second) {
17583106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        MacroGroup = MacroStack.back().first;
17593106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        break;
176061f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek      }
17611eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17623106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      MacroStack.pop_back();
17636837faa083bebad39aa342f84c2b450fb6410eafTed Kremenek    }
17641eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17653106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    if (!MacroGroup || ParentInstantiationLoc == MacroStack.back().second) {
17663106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      // Create a new macro group and add it to the stack.
1767590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks      PathDiagnosticMacroPiece *NewGroup =
1768590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks        new PathDiagnosticMacroPiece(
176977d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek          PathDiagnosticLocation::createSingleLocation(piece->getLocation()));
17705a429954aedd8a830a70fcea2cb51561673e4fc3Ted Kremenek
17713106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      if (MacroGroup)
1772802e02463b880f53a6e645bde78cc412481ce9e0Ted Kremenek        MacroGroup->subPieces.push_back(NewGroup);
17733106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      else {
17743106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        assert(InstantiationLoc.isFileID());
17753106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek        Pieces.push_back(NewGroup);
17763106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      }
17771eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17783106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      MacroGroup = NewGroup;
17793106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      MacroStack.push_back(std::make_pair(MacroGroup, InstantiationLoc));
17807dc8664a54f4ede40a5f4adee3f5081a59d7ee1cTed Kremenek    }
17813106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek
17823106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek    // Finally, add the PathDiagnosticPiece to the group.
178377d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek    MacroGroup->subPieces.push_back(piece);
17843106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  }
17851eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
17863106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Now take the pieces and construct a new PathDiagnostic.
178777d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek  path.clear();
17881eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
178977d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek  for (PiecesTy::iterator I=Pieces.begin(), E=Pieces.end(); I!=E; ++I)
179077d09441e59d3bced6c3d55505eb3a67a784fe02Ted Kremenek    path.push_back(*I);
179161f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek}
179261f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek
17933106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenekvoid GRBugReporter::GeneratePathDiagnostic(PathDiagnostic& PD,
17945f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner                        SmallVectorImpl<BugReport *> &bugReports) {
17951eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
179640406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  assert(!bugReports.empty());
17975f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner  SmallVector<const ExplodedNode *, 10> errorNodes;
17985f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner  for (SmallVectorImpl<BugReport*>::iterator I = bugReports.begin(),
179940406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    E = bugReports.end(); I != E; ++I) {
180040406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek      errorNodes.push_back((*I)->getErrorNode());
180140406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  }
18021eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
18033106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Construct a new graph that contains only a single path from the error
18041eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  // node to a root.
180538b02b912e1a55c912f603c4369431264d36a381Zhongxing Xu  const std::pair<std::pair<ExplodedGraph*, NodeBackMap*>,
1806c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu  std::pair<ExplodedNode*, unsigned> >&
180740406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    GPair = MakeReportGraph(&getGraph(), errorNodes);
18081eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
18093106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  // Find the BugReport with the original location.
181040406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  assert(GPair.second.second < bugReports.size());
181140406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  BugReport *R = bugReports[GPair.second.second];
18123106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek  assert(R && "No original report found for sliced graph.");
18131eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
18146f42b62b6194f53bcbc349f5d17388e1936535d7Dylan Noblesmith  OwningPtr<ExplodedGraph> ReportGraph(GPair.first.first);
18156f42b62b6194f53bcbc349f5d17388e1936535d7Dylan Noblesmith  OwningPtr<NodeBackMap> BackMap(GPair.first.second);
1816c5619d901a68dc27a9e310a6a831f03efebcd950Zhongxing Xu  const ExplodedNode *N = GPair.second.first;
18171eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
18181eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  // Start building the path diagnostic...
1819ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie  PathDiagnosticBuilder PDB(*this, R, BackMap.get(),
1820ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie                            getPathDiagnosticConsumer());
18211eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
18228e6431adab313e283a992698f6fc7afe62420999Anna Zaks  // Register additional node visitors.
182350bbc165b063155cc23c360deb7b865502e068e2Anna Zaks  R->addVisitor(new NilReceiverBRVisitor());
182450bbc165b063155cc23c360deb7b865502e068e2Anna Zaks  R->addVisitor(new ConditionBRVisitor());
18251eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
18263bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  BugReport::VisitorList visitors;
18273bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  unsigned originalReportConfigToken, finalReportConfigToken;
18283bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose
18293bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  // While generating diagnostics, it's possible the visitors will decide
18303bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  // new symbols and regions are interesting, or add other visitors based on
18313bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  // the information they find. If they do, we need to regenerate the path
18323bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  // based on our new report configuration.
18333bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  do {
18343bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    // Get a clean copy of all the visitors.
18353bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    for (BugReport::visitor_iterator I = R->visitor_begin(),
18363bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose                                     E = R->visitor_end(); I != E; ++I)
18373bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose       visitors.push_back((*I)->clone());
18383bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose
18393bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    // Clear out the active path from any previous work.
18403bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    PD.getActivePath().clear();
18413bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    originalReportConfigToken = R->getConfigurationChangeToken();
18423bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose
18433bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    // Generate the very last diagnostic piece - the piece is visible before
18443bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    // the trace is expanded.
18453bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    PathDiagnosticPiece *LastPiece = 0;
18463bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    for (BugReport::visitor_iterator I = visitors.begin(), E = visitors.end();
18473bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose         I != E; ++I) {
18483bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose      if (PathDiagnosticPiece *Piece = (*I)->getEndPath(PDB, N, *R)) {
18493bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose        assert (!LastPiece &&
18503bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose                "There can only be one final piece in a diagnostic.");
18513bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose        LastPiece = Piece;
18523bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose      }
185323f395ee1bf4e4aa76b310d896a951799eaca94aAnna Zaks    }
18543bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    if (!LastPiece)
18553bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose      LastPiece = BugReporterVisitor::getDefaultEndPath(PDB, N, *R);
18563bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    if (LastPiece)
18573bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose      PD.getActivePath().push_back(LastPiece);
18583bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    else
18593bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose      return;
186023f395ee1bf4e4aa76b310d896a951799eaca94aAnna Zaks
18613bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    switch (PDB.getGenerationScheme()) {
1862ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie    case PathDiagnosticConsumer::Extensive:
18633bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose      GenerateExtensivePathDiagnostic(PD, PDB, N, visitors);
18645fb5dfb6646464db3cd6d54a6332375c8fe36b75Ted Kremenek      break;
1865ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie    case PathDiagnosticConsumer::Minimal:
18663bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose      GenerateMinimalPathDiagnostic(PD, PDB, N, visitors);
18673106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek      break;
18683bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    }
18693bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose
18703bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    // Clean up the visitors we used.
18713bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    llvm::DeleteContainerPointers(visitors);
18723bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose
18733bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    // Did anything change while generating this path?
18743bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose    finalReportConfigToken = R->getConfigurationChangeToken();
18753bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose  } while(finalReportConfigToken != originalReportConfigToken);
18763bc75ca0a636efdc93471c9b6bad43085a22bf3aJordy Rose
1877c89f4b05721f53cfbaf32fc0c4919a4616e68440Ted Kremenek  // Finally, prune the diagnostic path of uninteresting stuff.
1878ed7948b55fa4b2505f240cc5287137f451172b4cTed Kremenek  if (R->shouldPrunePath()) {
1879ed7948b55fa4b2505f240cc5287137f451172b4cTed Kremenek    bool hasSomethingInteresting = RemoveUneededCalls(PD.getMutablePieces());
1880ed7948b55fa4b2505f240cc5287137f451172b4cTed Kremenek    assert(hasSomethingInteresting);
1881ed7948b55fa4b2505f240cc5287137f451172b4cTed Kremenek    (void) hasSomethingInteresting;
1882ed7948b55fa4b2505f240cc5287137f451172b4cTed Kremenek  }
18833106198188ac6eb2753f1764d5c28376b0b76351Ted Kremenek}
18841aa44c7c9198796085b77c6bdd7c90030845a688Ted Kremenek
1885cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenekvoid BugReporter::Register(BugType *BT) {
18863baf672378f105602d2b12f03f00277ae1936fe9Ted Kremenek  BugTypes = F.add(BugTypes, BT);
188776d90c8bf83d071fd1bdbffa6d9f2af87c6bb7b5Ted Kremenek}
188876d90c8bf83d071fd1bdbffa6d9f2af87c6bb7b5Ted Kremenek
18891eb4433ac451dc16f4133a88af2d002ac26c58efMike Stumpvoid BugReporter::EmitReport(BugReport* R) {
1890cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  // Compute the bug report's hash to determine its equivalence class.
1891cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  llvm::FoldingSetNodeID ID;
1892cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  R->Profile(ID);
18931eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
18941eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  // Lookup the equivance class.  If there isn't one, create it.
1895cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  BugType& BT = R->getBugType();
1896cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  Register(&BT);
1897cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  void *InsertPos;
1898404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  BugReportEquivClass* EQ = EQClasses.FindNodeOrInsertPos(ID, InsertPos);
18991eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1900cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  if (!EQ) {
1901cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek    EQ = new BugReportEquivClass(R);
1902404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis    EQClasses.InsertNode(EQ, InsertPos);
19033b030a28cda2b953758507769c1d436bec5ec45eAnna Zaks    EQClassesVector.push_back(EQ);
1904cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  }
1905cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  else
1906cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek    EQ->AddReport(R);
190761f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek}
190861f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek
190906c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
191006c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek//===----------------------------------------------------------------------===//
191106c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek// Emitting reports in equivalence classes.
191206c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek//===----------------------------------------------------------------------===//
191306c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
191406c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremeneknamespace {
1915ba5fb5a955c896815c439289fc51c03cf0635129Kovarththanan Rajaratnamstruct FRIEC_WLItem {
191606c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  const ExplodedNode *N;
191706c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  ExplodedNode::const_succ_iterator I, E;
191806c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
191906c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  FRIEC_WLItem(const ExplodedNode *n)
192006c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  : N(n), I(N->succ_begin()), E(N->succ_end()) {}
192106c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek};
192206c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek}
192306c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
192461f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenekstatic BugReport *
192561f52bd3c524268e25b48a1ed3730aedd6cc8374Ted KremenekFindReportInEquivalenceClass(BugReportEquivClass& EQ,
19265f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner                             SmallVectorImpl<BugReport*> &bugReports) {
192761f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek
192806c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  BugReportEquivClass::iterator I = EQ.begin(), E = EQ.end();
192906c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  assert(I != E);
19304a5f724538cbc275370c9504e8169ce92503256cBenjamin Kramer  BugType& BT = I->getBugType();
193161f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek
193240406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  // If we don't need to suppress any of the nodes because they are
193340406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  // post-dominated by a sink, simply add all the nodes in the equivalence class
193440406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  // to 'Nodes'.  Any of the reports will serve as a "representative" report.
193561f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek  if (!BT.isSuppressOnSink()) {
19364a5f724538cbc275370c9504e8169ce92503256cBenjamin Kramer    BugReport *R = I;
193761f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek    for (BugReportEquivClass::iterator I=EQ.begin(), E=EQ.end(); I!=E; ++I) {
19389c378f705405d37f49795d5e915989de774fe11fTed Kremenek      const ExplodedNode *N = I->getErrorNode();
193961f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek      if (N) {
19404a5f724538cbc275370c9504e8169ce92503256cBenjamin Kramer        R = I;
194140406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek        bugReports.push_back(R);
194261f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek      }
194361f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek    }
194406c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    return R;
194561f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek  }
194661f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek
194706c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  // For bug reports that should be suppressed when all paths are post-dominated
194806c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  // by a sink node, iterate through the reports in the equivalence class
194906c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  // until we find one that isn't post-dominated (if one exists).  We use a
195006c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  // DFS traversal of the ExplodedGraph to find a non-sink node.  We could write
195106c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  // this as a recursive function, but we don't want to risk blowing out the
195206c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  // stack for very long paths.
195340406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  BugReport *exampleReport = 0;
195461f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek
195506c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  for (; I != E; ++I) {
19564a5f724538cbc275370c9504e8169ce92503256cBenjamin Kramer    const ExplodedNode *errorNode = I->getErrorNode();
195706c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
195840406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    if (!errorNode)
195906c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek      continue;
196040406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    if (errorNode->isSink()) {
1961b219cfc4d75f0a03630b7c4509ef791b7e97b2c8David Blaikie      llvm_unreachable(
196206c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek           "BugType::isSuppressSink() should not be 'true' for sink end nodes");
196306c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    }
196461f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek    // No successors?  By definition this nodes isn't post-dominated by a sink.
196540406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    if (errorNode->succ_empty()) {
19664a5f724538cbc275370c9504e8169ce92503256cBenjamin Kramer      bugReports.push_back(I);
196740406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek      if (!exampleReport)
19684a5f724538cbc275370c9504e8169ce92503256cBenjamin Kramer        exampleReport = I;
196961f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek      continue;
197061f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek    }
197161f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek
197206c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    // At this point we know that 'N' is not a sink and it has at least one
197306c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    // successor.  Use a DFS worklist to find a non-sink end-of-path node.
197406c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    typedef FRIEC_WLItem WLItem;
19755f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner    typedef SmallVector<WLItem, 10> DFSWorkList;
197606c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    llvm::DenseMap<const ExplodedNode *, unsigned> Visited;
197706c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
197806c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    DFSWorkList WL;
197940406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    WL.push_back(errorNode);
198040406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    Visited[errorNode] = 1;
198106c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
198206c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    while (!WL.empty()) {
198306c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek      WLItem &WI = WL.back();
198406c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek      assert(!WI.N->succ_empty());
198506c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
198606c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek      for (; WI.I != WI.E; ++WI.I) {
198706c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        const ExplodedNode *Succ = *WI.I;
198806c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        // End-of-path node?
198906c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        if (Succ->succ_empty()) {
199061f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek          // If we found an end-of-path node that is not a sink.
199161f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek          if (!Succ->isSink()) {
19924a5f724538cbc275370c9504e8169ce92503256cBenjamin Kramer            bugReports.push_back(I);
199340406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek            if (!exampleReport)
19944a5f724538cbc275370c9504e8169ce92503256cBenjamin Kramer              exampleReport = I;
199561f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek            WL.clear();
199661f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek            break;
199761f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek          }
199806c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek          // Found a sink?  Continue on to the next successor.
199906c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek          continue;
200006c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        }
200106c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        // Mark the successor as visited.  If it hasn't been explored,
200206c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        // enqueue it to the DFS worklist.
200306c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        unsigned &mark = Visited[Succ];
200406c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        if (!mark) {
200506c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek          mark = 1;
200606c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek          WL.push_back(Succ);
200706c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek          break;
200806c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        }
200906c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek      }
201061f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek
201161f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek      // The worklist may have been cleared at this point.  First
201261f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek      // check if it is empty before checking the last item.
201361f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek      if (!WL.empty() && &WL.back() == &WI)
201406c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek        WL.pop_back();
201506c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    }
201606c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek  }
201706c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
201861f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek  // ExampleReport will be NULL if all the nodes in the equivalence class
201961f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek  // were post-dominated by sinks.
202040406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  return exampleReport;
202161f52bd3c524268e25b48a1ed3730aedd6cc8374Ted Kremenek}
2022e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2023e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek//===----------------------------------------------------------------------===//
2024e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek// DiagnosticCache.  This is a hack to cache analyzer diagnostics.  It
2025e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek// uses global state, which eventually should go elsewhere.
2026e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek//===----------------------------------------------------------------------===//
2027e0a58073b76fc016325a35152533b8468df2bf4aTed Kremeneknamespace {
2028ba5fb5a955c896815c439289fc51c03cf0635129Kovarththanan Rajaratnamclass DiagCacheItem : public llvm::FoldingSetNode {
2029e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  llvm::FoldingSetNodeID ID;
2030e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenekpublic:
2031e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  DiagCacheItem(BugReport *R, PathDiagnostic *PD) {
20324522e2a9e7fa0313e8e5a388d8f0ab66feccc6afAnna Zaks    R->Profile(ID);
20334522e2a9e7fa0313e8e5a388d8f0ab66feccc6afAnna Zaks    PD->Profile(ID);
2034e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  }
2035e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2036e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  void Profile(llvm::FoldingSetNodeID &id) {
2037e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek    id = ID;
2038e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  }
2039e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2040e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  llvm::FoldingSetNodeID &getID() { return ID; }
2041e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek};
2042e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek}
2043e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2044e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenekstatic bool IsCachedDiagnostic(BugReport *R, PathDiagnostic *PD) {
2045e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  // FIXME: Eventually this diagnostic cache should reside in something
2046e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  // like AnalysisManager instead of being a static variable.  This is
2047e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  // really unsafe in the long term.
2048e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  typedef llvm::FoldingSet<DiagCacheItem> DiagnosticCache;
2049e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  static DiagnosticCache DC;
2050e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2051e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  void *InsertPos;
2052e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  DiagCacheItem *Item = new DiagCacheItem(R, PD);
2053e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2054e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  if (DC.FindNodeOrInsertPos(Item->getID(), InsertPos)) {
2055e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek    delete Item;
2056e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek    return true;
2057e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  }
2058e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2059e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  DC.InsertNode(Item, InsertPos);
2060e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek  return false;
2061e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek}
2062e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2063cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenekvoid BugReporter::FlushReport(BugReportEquivClass& EQ) {
20645f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner  SmallVector<BugReport*, 10> bugReports;
206540406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  BugReport *exampleReport = FindReportInEquivalenceClass(EQ, bugReports);
206640406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  if (!exampleReport)
206706c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek    return;
206806c9cb4d1a29e708f51bce9f7ee5acbe1c3761c3Ted Kremenek
2069ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie  PathDiagnosticConsumer* PD = getPathDiagnosticConsumer();
20701eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2071cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  // FIXME: Make sure we use the 'R' for the path that was actually used.
20721eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  // Probably doesn't make a difference in practice.
207340406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  BugType& BT = exampleReport->getBugType();
20741eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
20756f42b62b6194f53bcbc349f5d17388e1936535d7Dylan Noblesmith  OwningPtr<PathDiagnostic>
207607189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek    D(new PathDiagnostic(exampleReport->getDeclWithIssue(),
207707189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek                         exampleReport->getBugType().getName(),
2078da0e8429a3598acfdd0ecf15135d432e4dd3517cTed Kremenek                         !PD || PD->useVerboseDescription()
207940406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek                         ? exampleReport->getDescription()
208040406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek                         : exampleReport->getShortDescription(),
2081d49967f8764135ae65658e354b6d38e3637c9de3Ted Kremenek                         BT.getCategory()));
2082d49967f8764135ae65658e354b6d38e3637c9de3Ted Kremenek
208340406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek  if (!bugReports.empty())
208440406fe63df2b932d6e9fd021f77f097f9d33afbTed Kremenek    GeneratePathDiagnostic(*D.get(), bugReports);
2085e0a58073b76fc016325a35152533b8468df2bf4aTed Kremenek
2086072192bcbb05a0fee7ec3061750b27e8d2004952Ted Kremenek  // Get the meta data.
20877f2531cb41448852ec78de90fc1d3c0149c95d7dAnna Zaks  const BugReport::ExtraTextList &Meta =
20887f2531cb41448852ec78de90fc1d3c0149c95d7dAnna Zaks                                  exampleReport->getExtraText();
20897f2531cb41448852ec78de90fc1d3c0149c95d7dAnna Zaks  for (BugReport::ExtraTextList::const_iterator i = Meta.begin(),
20907f2531cb41448852ec78de90fc1d3c0149c95d7dAnna Zaks                                                e = Meta.end(); i != e; ++i) {
20917f2531cb41448852ec78de90fc1d3c0149c95d7dAnna Zaks    D->addMeta(*i);
20927f2531cb41448852ec78de90fc1d3c0149c95d7dAnna Zaks  }
209375840e1501563fe7c3dcb5600b75965ba1fe1bc4Ted Kremenek
20943148eb4a75f70f2636075c364d03104223f004d3Ted Kremenek  // Emit a summary diagnostic to the regular Diagnostics engine.
2095640ccf071076e684713cc3c3276bb51982bff607Argyrios Kyrtzidis  BugReport::ranges_iterator Beg, End;
2096640ccf071076e684713cc3c3276bb51982bff607Argyrios Kyrtzidis  llvm::tie(Beg, End) = exampleReport->getRanges();
2097d6471f7c1921c7802804ce3ff6fe9768310f72b9David Blaikie  DiagnosticsEngine &Diag = getDiagnostic();
2098c213b48206c55ca0eb1387cfa1651de504f147d1Ted Kremenek
209988fc18120ca14b82bef695d6440f51e4c468916cTed Kremenek  if (!IsCachedDiagnostic(exampleReport, D.get())) {
210088fc18120ca14b82bef695d6440f51e4c468916cTed Kremenek    // Search the description for '%', as that will be interpretted as a
210188fc18120ca14b82bef695d6440f51e4c468916cTed Kremenek    // format character by FormatDiagnostics.
210288fc18120ca14b82bef695d6440f51e4c468916cTed Kremenek    StringRef desc = exampleReport->getShortDescription();
210388fc18120ca14b82bef695d6440f51e4c468916cTed Kremenek
2104f7ccbad5d9949e7ddd1cbef43d482553b811e026Dylan Noblesmith    SmallString<512> TmpStr;
2105c213b48206c55ca0eb1387cfa1651de504f147d1Ted Kremenek    llvm::raw_svector_ostream Out(TmpStr);
210688fc18120ca14b82bef695d6440f51e4c468916cTed Kremenek    for (StringRef::iterator I=desc.begin(), E=desc.end(); I!=E; ++I) {
2107c213b48206c55ca0eb1387cfa1651de504f147d1Ted Kremenek      if (*I == '%')
2108c213b48206c55ca0eb1387cfa1651de504f147d1Ted Kremenek        Out << "%%";
2109c213b48206c55ca0eb1387cfa1651de504f147d1Ted Kremenek      else
2110c213b48206c55ca0eb1387cfa1651de504f147d1Ted Kremenek        Out << *I;
211188fc18120ca14b82bef695d6440f51e4c468916cTed Kremenek    }
2112c213b48206c55ca0eb1387cfa1651de504f147d1Ted Kremenek
2113c213b48206c55ca0eb1387cfa1651de504f147d1Ted Kremenek    Out.flush();
211488fc18120ca14b82bef695d6440f51e4c468916cTed Kremenek    unsigned ErrorDiag = Diag.getCustomDiagID(DiagnosticsEngine::Warning, TmpStr);
21150a14eee528a901c16f0e288fbc10a3abc1660d87Chris Lattner
2116590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks    DiagnosticBuilder diagBuilder = Diag.Report(
2117590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks      exampleReport->getLocation(getSourceManager()).asLocation(), ErrorDiag);
2118640ccf071076e684713cc3c3276bb51982bff607Argyrios Kyrtzidis    for (BugReport::ranges_iterator I = Beg; I != End; ++I)
2119b6b7e7bff3101062521de6e79533a3c25e2195d9Argyrios Kyrtzidis      diagBuilder << *I;
21202f0e89ea96292d2974eb1a7dddc0e9870aa86bb7Ted Kremenek  }
21213148eb4a75f70f2636075c364d03104223f004d3Ted Kremenek
2122ef3643fbbbf66247c5e205497fae0f46e240c143David Blaikie  // Emit a full diagnostic for the path if we have a PathDiagnosticConsumer.
21233148eb4a75f70f2636075c364d03104223f004d3Ted Kremenek  if (!PD)
21243148eb4a75f70f2636075c364d03104223f004d3Ted Kremenek    return;
21251eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2126802e02463b880f53a6e645bde78cc412481ce9e0Ted Kremenek  if (D->path.empty()) {
2127590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks    PathDiagnosticPiece *piece = new PathDiagnosticEventPiece(
2128590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks                                 exampleReport->getLocation(getSourceManager()),
2129590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks                                 exampleReport->getDescription());
213007189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek    for ( ; Beg != End; ++Beg)
213107189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek      piece->addRange(*Beg);
21321fbfd5b9b8b82aea084773b76dd1ec6796a7000cTed Kremenek
21332042fc1f36d471f437023e8899f0c4fadded2341Ted Kremenek    D->getActivePath().push_back(piece);
21343148eb4a75f70f2636075c364d03104223f004d3Ted Kremenek  }
21351eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
21363148eb4a75f70f2636075c364d03104223f004d3Ted Kremenek  PD->HandlePathDiagnostic(D.take());
213761f3e058056ab519d249aa67e3d52b0ead57c63eTed Kremenek}
213857202071e477530e9348bc76671ee369b2399b92Ted Kremenek
213907189521a15d9c088216b943649cb9fe231cbb57Ted Kremenekvoid BugReporter::EmitBasicReport(const Decl *DeclWithIssue,
214007189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek                                  StringRef name,
21415f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner                                  StringRef category,
2142590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks                                  StringRef str, PathDiagnosticLocation Loc,
21438c036c7f77d69f96df49219ed0bdbade200d52ebTed Kremenek                                  SourceRange* RBeg, unsigned NumRanges) {
21441eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2145404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  // 'BT' is owned by BugReporter.
2146404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  BugType *BT = getBugTypeForName(name, category);
2147590dd8e0959d8df5621827768987c4792b74fc06Anna Zaks  BugReport *R = new BugReport(*BT, str, Loc);
214807189521a15d9c088216b943649cb9fe231cbb57Ted Kremenek  R->setDeclWithIssue(DeclWithIssue);
2149cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  for ( ; NumRanges > 0 ; --NumRanges, ++RBeg) R->addRange(*RBeg);
2150cf118d41f7930a18dce97416ef7834a62642f587Ted Kremenek  EmitReport(R);
2151cabe66811fe43835b8c5a0854552768fc53261e3Ted Kremenek}
2152404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis
21535f9e272e632e951b1efe824cd16acb4d96077930Chris LattnerBugType *BugReporter::getBugTypeForName(StringRef name,
21545f9e272e632e951b1efe824cd16acb4d96077930Chris Lattner                                        StringRef category) {
2155f7ccbad5d9949e7ddd1cbef43d482553b811e026Dylan Noblesmith  SmallString<136> fullDesc;
2156404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  llvm::raw_svector_ostream(fullDesc) << name << ":" << category;
2157404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  llvm::StringMapEntry<BugType *> &
2158404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis      entry = StrBugTypes.GetOrCreateValue(fullDesc);
2159404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  BugType *BT = entry.getValue();
2160404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  if (!BT) {
2161404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis    BT = new BugType(name, category);
2162404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis    entry.setValue(BT);
2163404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  }
2164404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis  return BT;
2165404fc3ad6bd844bf8ce70cbf9974ab297704a122Argyrios Kyrtzidis}
2166