110e25031STed Kremenek //===- ExprEngineCXX.cpp - ExprEngine support for C++ -----------*- C++ -*-===// 210e25031STed Kremenek // 310e25031STed Kremenek // The LLVM Compiler Infrastructure 410e25031STed Kremenek // 510e25031STed Kremenek // This file is distributed under the University of Illinois Open Source 610e25031STed Kremenek // License. See LICENSE.TXT for details. 710e25031STed Kremenek // 810e25031STed Kremenek //===----------------------------------------------------------------------===// 910e25031STed Kremenek // 1010e25031STed Kremenek // This file defines the C++ expression evaluation engine. 1110e25031STed Kremenek // 1210e25031STed Kremenek //===----------------------------------------------------------------------===// 1310e25031STed Kremenek 1410e25031STed Kremenek #include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h" 1510e25031STed Kremenek #include "clang/AST/DeclCXX.h" 168fdb59f9STed Kremenek #include "clang/AST/StmtCXX.h" 17*e8ba3ec4SArtem Dergachev #include "clang/AST/ParentMap.h" 1817a8757aSJordan Rose #include "clang/Basic/PrettyStackTrace.h" 193a02247dSChandler Carruth #include "clang/StaticAnalyzer/Core/CheckerManager.h" 203a02247dSChandler Carruth #include "clang/StaticAnalyzer/Core/PathSensitive/AnalysisManager.h" 213a02247dSChandler Carruth #include "clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h" 2210e25031STed Kremenek 2310e25031STed Kremenek using namespace clang; 2410e25031STed Kremenek using namespace ento; 2510e25031STed Kremenek 2610e25031STed Kremenek void ExprEngine::CreateCXXTemporaryObject(const MaterializeTemporaryExpr *ME, 2710e25031STed Kremenek ExplodedNode *Pred, 2810e25031STed Kremenek ExplodedNodeSet &Dst) { 29d94854a4STed Kremenek StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 302a14c695STed Kremenek const Expr *tempExpr = ME->GetTemporaryExpr()->IgnoreParens(); 3149b1e38eSTed Kremenek ProgramStateRef state = Pred->getState(); 32632e3b7eSTed Kremenek const LocationContext *LCtx = Pred->getLocationContext(); 3310e25031STed Kremenek 34861a1740SJordan Rose state = createTemporaryRegionIfNeeded(state, LCtx, tempExpr, ME); 355772f82dSJordan Rose Bldr.generateNode(ME, Pred, state); 3610e25031STed Kremenek } 3710e25031STed Kremenek 383d7b7f52SJordan Rose // FIXME: This is the sort of code that should eventually live in a Core 393d7b7f52SJordan Rose // checker rather than as a special case in ExprEngine. 4088bb563cSJordan Rose void ExprEngine::performTrivialCopy(NodeBuilder &Bldr, ExplodedNode *Pred, 413d7b7f52SJordan Rose const CallEvent &Call) { 423d7b7f52SJordan Rose SVal ThisVal; 433d7b7f52SJordan Rose bool AlwaysReturnsLValue; 443d7b7f52SJordan Rose if (const CXXConstructorCall *Ctor = dyn_cast<CXXConstructorCall>(&Call)) { 453d7b7f52SJordan Rose assert(Ctor->getDecl()->isTrivial()); 463d7b7f52SJordan Rose assert(Ctor->getDecl()->isCopyOrMoveConstructor()); 473d7b7f52SJordan Rose ThisVal = Ctor->getCXXThisVal(); 483d7b7f52SJordan Rose AlwaysReturnsLValue = false; 493d7b7f52SJordan Rose } else { 503d7b7f52SJordan Rose assert(cast<CXXMethodDecl>(Call.getDecl())->isTrivial()); 513d7b7f52SJordan Rose assert(cast<CXXMethodDecl>(Call.getDecl())->getOverloadedOperator() == 523d7b7f52SJordan Rose OO_Equal); 533d7b7f52SJordan Rose ThisVal = cast<CXXInstanceCall>(Call).getCXXThisVal(); 543d7b7f52SJordan Rose AlwaysReturnsLValue = true; 553d7b7f52SJordan Rose } 5688bb563cSJordan Rose 5788bb563cSJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 5888bb563cSJordan Rose 5988bb563cSJordan Rose ExplodedNodeSet Dst; 6088bb563cSJordan Rose Bldr.takeNodes(Pred); 6188bb563cSJordan Rose 6288bb563cSJordan Rose SVal V = Call.getArgSVal(0); 6388bb563cSJordan Rose 643d7b7f52SJordan Rose // If the value being copied is not unknown, load from its location to get 653d7b7f52SJordan Rose // an aggregate rvalue. 6605785d16SDavid Blaikie if (Optional<Loc> L = V.getAs<Loc>()) 6788bb563cSJordan Rose V = Pred->getState()->getSVal(*L); 683d7b7f52SJordan Rose else 6975f9d3acSArtem Dergachev assert(V.isUnknownOrUndef()); 7088bb563cSJordan Rose 713d7b7f52SJordan Rose const Expr *CallExpr = Call.getOriginExpr(); 723d7b7f52SJordan Rose evalBind(Dst, CallExpr, Pred, ThisVal, V, true); 7388bb563cSJordan Rose 743d7b7f52SJordan Rose PostStmt PS(CallExpr, LCtx); 7588bb563cSJordan Rose for (ExplodedNodeSet::iterator I = Dst.begin(), E = Dst.end(); 7688bb563cSJordan Rose I != E; ++I) { 7788bb563cSJordan Rose ProgramStateRef State = (*I)->getState(); 783d7b7f52SJordan Rose if (AlwaysReturnsLValue) 793d7b7f52SJordan Rose State = State->BindExpr(CallExpr, LCtx, ThisVal); 803d7b7f52SJordan Rose else 8188bb563cSJordan Rose State = bindReturnValue(Call, LCtx, State); 8288bb563cSJordan Rose Bldr.generateNode(PS, State, *I); 8388bb563cSJordan Rose } 8488bb563cSJordan Rose } 8588bb563cSJordan Rose 86bc74eb1cSJordan Rose 87bc74eb1cSJordan Rose /// Returns a region representing the first element of a (possibly 88bc74eb1cSJordan Rose /// multi-dimensional) array. 89bc74eb1cSJordan Rose /// 90bc74eb1cSJordan Rose /// On return, \p Ty will be set to the base type of the array. 91bc74eb1cSJordan Rose /// 92bc74eb1cSJordan Rose /// If the type is not an array type at all, the original value is returned. 93bc74eb1cSJordan Rose static SVal makeZeroElementRegion(ProgramStateRef State, SVal LValue, 94bc74eb1cSJordan Rose QualType &Ty) { 95bc74eb1cSJordan Rose SValBuilder &SVB = State->getStateManager().getSValBuilder(); 96bc74eb1cSJordan Rose ASTContext &Ctx = SVB.getContext(); 97bc74eb1cSJordan Rose 98bc74eb1cSJordan Rose while (const ArrayType *AT = Ctx.getAsArrayType(Ty)) { 99bc74eb1cSJordan Rose Ty = AT->getElementType(); 100bc74eb1cSJordan Rose LValue = State->getLValue(Ty, SVB.makeZeroArrayIndex(), LValue); 101bc74eb1cSJordan Rose } 102bc74eb1cSJordan Rose 103bc74eb1cSJordan Rose return LValue; 104bc74eb1cSJordan Rose } 105bc74eb1cSJordan Rose 10617f41608SJordan Rose 107412c0af2SDevin Coughlin const MemRegion * 108412c0af2SDevin Coughlin ExprEngine::getRegionForConstructedObject(const CXXConstructExpr *CE, 109412c0af2SDevin Coughlin ExplodedNode *Pred) { 110b970505dSJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 11105375eb4SJordan Rose ProgramStateRef State = Pred->getState(); 11205375eb4SJordan Rose 11305375eb4SJordan Rose // See if we're constructing an existing region by looking at the next 11405375eb4SJordan Rose // element in the CFG. 1158a5a094eSJordan Rose 116412c0af2SDevin Coughlin if (auto Elem = findElementDirectlyInitializedByCurrentConstructor()) { 117412c0af2SDevin Coughlin if (Optional<CFGStmt> StmtElem = Elem->getAs<CFGStmt>()) { 118412c0af2SDevin Coughlin auto *DS = cast<DeclStmt>(StmtElem->getStmt()); 119412c0af2SDevin Coughlin if (const auto *Var = dyn_cast<VarDecl>(DS->getSingleDecl())) { 120bcd88973SJordan Rose if (Var->getInit() && Var->getInit()->IgnoreImplicit() == CE) { 121bc74eb1cSJordan Rose SVal LValue = State->getLValue(Var, LCtx); 12220edae87SJordan Rose QualType Ty = Var->getType(); 123bc74eb1cSJordan Rose LValue = makeZeroElementRegion(State, LValue, Ty); 12417f41608SJordan Rose return LValue.getAsRegion(); 12520edae87SJordan Rose } 12620edae87SJordan Rose } 127412c0af2SDevin Coughlin } else if (Optional<CFGInitializer> InitElem = Elem->getAs<CFGInitializer>()) { 12800be69abSDavid Blaikie const CXXCtorInitializer *Init = InitElem->getInitializer(); 12954529a34SJordan Rose assert(Init->isAnyMemberInitializer()); 13054529a34SJordan Rose const CXXMethodDecl *CurCtor = cast<CXXMethodDecl>(LCtx->getDecl()); 131412c0af2SDevin Coughlin Loc ThisPtr = 132412c0af2SDevin Coughlin getSValBuilder().getCXXThis(CurCtor, LCtx->getCurrentStackFrame()); 13354529a34SJordan Rose SVal ThisVal = State->getSVal(ThisPtr); 13454529a34SJordan Rose 135bc74eb1cSJordan Rose const ValueDecl *Field; 136bc74eb1cSJordan Rose SVal FieldVal; 13754529a34SJordan Rose if (Init->isIndirectMemberInitializer()) { 138bc74eb1cSJordan Rose Field = Init->getIndirectMember(); 139bc74eb1cSJordan Rose FieldVal = State->getLValue(Init->getIndirectMember(), ThisVal); 14054529a34SJordan Rose } else { 141bc74eb1cSJordan Rose Field = Init->getMember(); 142bc74eb1cSJordan Rose FieldVal = State->getLValue(Init->getMember(), ThisVal); 14354529a34SJordan Rose } 144bc74eb1cSJordan Rose 145bc74eb1cSJordan Rose QualType Ty = Field->getType(); 146bc74eb1cSJordan Rose FieldVal = makeZeroElementRegion(State, FieldVal, Ty); 14717f41608SJordan Rose return FieldVal.getAsRegion(); 14854529a34SJordan Rose } 14954529a34SJordan Rose 15005375eb4SJordan Rose // FIXME: This will eventually need to handle new-expressions as well. 15117f41608SJordan Rose // Don't forget to update the pre-constructor initialization code in 15217f41608SJordan Rose // ExprEngine::VisitCXXConstructExpr. 15305375eb4SJordan Rose } 154c9318304SJordan Rose // If we couldn't find an existing region to construct into, assume we're 155c9318304SJordan Rose // constructing a temporary. 156412c0af2SDevin Coughlin MemRegionManager &MRMgr = getSValBuilder().getRegionManager(); 15717f41608SJordan Rose return MRMgr.getCXXTempObjectRegion(CE, LCtx); 158c9318304SJordan Rose } 15905375eb4SJordan Rose 160412c0af2SDevin Coughlin /// Returns true if the initializer for \Elem can be a direct 161412c0af2SDevin Coughlin /// constructor. 162412c0af2SDevin Coughlin static bool canHaveDirectConstructor(CFGElement Elem){ 163412c0af2SDevin Coughlin // DeclStmts and CXXCtorInitializers for fields can be directly constructed. 164412c0af2SDevin Coughlin 165412c0af2SDevin Coughlin if (Optional<CFGStmt> StmtElem = Elem.getAs<CFGStmt>()) { 166412c0af2SDevin Coughlin if (isa<DeclStmt>(StmtElem->getStmt())) { 167412c0af2SDevin Coughlin return true; 168412c0af2SDevin Coughlin } 169412c0af2SDevin Coughlin } 170412c0af2SDevin Coughlin 171412c0af2SDevin Coughlin if (Elem.getKind() == CFGElement::Initializer) { 172412c0af2SDevin Coughlin return true; 173412c0af2SDevin Coughlin } 174412c0af2SDevin Coughlin 175412c0af2SDevin Coughlin return false; 176412c0af2SDevin Coughlin } 177412c0af2SDevin Coughlin 178412c0af2SDevin Coughlin Optional<CFGElement> 179412c0af2SDevin Coughlin ExprEngine::findElementDirectlyInitializedByCurrentConstructor() { 180412c0af2SDevin Coughlin const NodeBuilderContext &CurrBldrCtx = getBuilderContext(); 181412c0af2SDevin Coughlin // See if we're constructing an existing region by looking at the next 182412c0af2SDevin Coughlin // element in the CFG. 183412c0af2SDevin Coughlin const CFGBlock *B = CurrBldrCtx.getBlock(); 184412c0af2SDevin Coughlin assert(isa<CXXConstructExpr>(((*B)[currStmtIdx]).castAs<CFGStmt>().getStmt())); 185412c0af2SDevin Coughlin unsigned int NextStmtIdx = currStmtIdx + 1; 186412c0af2SDevin Coughlin if (NextStmtIdx >= B->size()) 187412c0af2SDevin Coughlin return None; 188412c0af2SDevin Coughlin 189412c0af2SDevin Coughlin CFGElement Next = (*B)[NextStmtIdx]; 190412c0af2SDevin Coughlin 191412c0af2SDevin Coughlin // Is this a destructor? If so, we might be in the middle of an assignment 192412c0af2SDevin Coughlin // to a local or member: look ahead one more element to see what we find. 193412c0af2SDevin Coughlin while (Next.getAs<CFGImplicitDtor>() && NextStmtIdx + 1 < B->size()) { 194412c0af2SDevin Coughlin ++NextStmtIdx; 195412c0af2SDevin Coughlin Next = (*B)[NextStmtIdx]; 196412c0af2SDevin Coughlin } 197412c0af2SDevin Coughlin 198412c0af2SDevin Coughlin if (canHaveDirectConstructor(Next)) 199412c0af2SDevin Coughlin return Next; 200412c0af2SDevin Coughlin 201412c0af2SDevin Coughlin return None; 202412c0af2SDevin Coughlin } 203412c0af2SDevin Coughlin 204412c0af2SDevin Coughlin const CXXConstructExpr * 205412c0af2SDevin Coughlin ExprEngine::findDirectConstructorForCurrentCFGElement() { 206412c0af2SDevin Coughlin // Go backward in the CFG to see if the previous element (ignoring 207412c0af2SDevin Coughlin // destructors) was a CXXConstructExpr. If so, that constructor 208412c0af2SDevin Coughlin // was constructed directly into an existing region. 209412c0af2SDevin Coughlin // This process is essentially the inverse of that performed in 210412c0af2SDevin Coughlin // findElementDirectlyInitializedByCurrentConstructor(). 211412c0af2SDevin Coughlin if (currStmtIdx == 0) 212412c0af2SDevin Coughlin return nullptr; 213412c0af2SDevin Coughlin 214412c0af2SDevin Coughlin const CFGBlock *B = getBuilderContext().getBlock(); 215412c0af2SDevin Coughlin assert(canHaveDirectConstructor((*B)[currStmtIdx])); 216412c0af2SDevin Coughlin 217412c0af2SDevin Coughlin unsigned int PreviousStmtIdx = currStmtIdx - 1; 218412c0af2SDevin Coughlin CFGElement Previous = (*B)[PreviousStmtIdx]; 219412c0af2SDevin Coughlin 220412c0af2SDevin Coughlin while (Previous.getAs<CFGImplicitDtor>() && PreviousStmtIdx > 0) { 221412c0af2SDevin Coughlin --PreviousStmtIdx; 222412c0af2SDevin Coughlin Previous = (*B)[PreviousStmtIdx]; 223412c0af2SDevin Coughlin } 224412c0af2SDevin Coughlin 225412c0af2SDevin Coughlin if (Optional<CFGStmt> PrevStmtElem = Previous.getAs<CFGStmt>()) { 226412c0af2SDevin Coughlin if (auto *CtorExpr = dyn_cast<CXXConstructExpr>(PrevStmtElem->getStmt())) { 227412c0af2SDevin Coughlin return CtorExpr; 228412c0af2SDevin Coughlin } 229412c0af2SDevin Coughlin } 230412c0af2SDevin Coughlin 231412c0af2SDevin Coughlin return nullptr; 232412c0af2SDevin Coughlin } 233412c0af2SDevin Coughlin 23417f41608SJordan Rose void ExprEngine::VisitCXXConstructExpr(const CXXConstructExpr *CE, 23517f41608SJordan Rose ExplodedNode *Pred, 23617f41608SJordan Rose ExplodedNodeSet &destNodes) { 23717f41608SJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 23817f41608SJordan Rose ProgramStateRef State = Pred->getState(); 23917f41608SJordan Rose 2400dbb783cSCraig Topper const MemRegion *Target = nullptr; 24117f41608SJordan Rose 24217f41608SJordan Rose // FIXME: Handle arrays, which run the same constructor for every element. 24317f41608SJordan Rose // For now, we just run the first constructor (which should still invalidate 24417f41608SJordan Rose // the entire array). 24517f41608SJordan Rose 24617f41608SJordan Rose switch (CE->getConstructionKind()) { 24717f41608SJordan Rose case CXXConstructExpr::CK_Complete: { 248412c0af2SDevin Coughlin Target = getRegionForConstructedObject(CE, Pred); 249b970505dSJordan Rose break; 250b970505dSJordan Rose } 251b3b976f0SJordan Rose case CXXConstructExpr::CK_VirtualBase: 252b3b976f0SJordan Rose // Make sure we are not calling virtual base class initializers twice. 253b3b976f0SJordan Rose // Only the most-derived object should initialize virtual base classes. 254b3b976f0SJordan Rose if (const Stmt *Outer = LCtx->getCurrentStackFrame()->getCallSite()) { 255b3b976f0SJordan Rose const CXXConstructExpr *OuterCtor = dyn_cast<CXXConstructExpr>(Outer); 256b3b976f0SJordan Rose if (OuterCtor) { 257b3b976f0SJordan Rose switch (OuterCtor->getConstructionKind()) { 258b970505dSJordan Rose case CXXConstructExpr::CK_NonVirtualBase: 259b970505dSJordan Rose case CXXConstructExpr::CK_VirtualBase: 260b3b976f0SJordan Rose // Bail out! 261b3b976f0SJordan Rose destNodes.Add(Pred); 262b3b976f0SJordan Rose return; 263b3b976f0SJordan Rose case CXXConstructExpr::CK_Complete: 264b3b976f0SJordan Rose case CXXConstructExpr::CK_Delegating: 265b3b976f0SJordan Rose break; 266b3b976f0SJordan Rose } 267b3b976f0SJordan Rose } 268b3b976f0SJordan Rose } 269b3b976f0SJordan Rose // FALLTHROUGH 270b3b976f0SJordan Rose case CXXConstructExpr::CK_NonVirtualBase: 271*e8ba3ec4SArtem Dergachev // In C++17, classes with non-virtual bases may be aggregates, so they would 272*e8ba3ec4SArtem Dergachev // be initialized as aggregates without a constructor call, so we may have 273*e8ba3ec4SArtem Dergachev // a base class constructed directly into an initializer list without 274*e8ba3ec4SArtem Dergachev // having the derived-class constructor call on the previous stack frame. 275*e8ba3ec4SArtem Dergachev // Initializer lists may be nested into more initializer lists that 276*e8ba3ec4SArtem Dergachev // correspond to surrounding aggregate initializations. 277*e8ba3ec4SArtem Dergachev // FIXME: For now this code essentially bails out. We need to find the 278*e8ba3ec4SArtem Dergachev // correct target region and set it. 279*e8ba3ec4SArtem Dergachev // FIXME: Instead of relying on the ParentMap, we should have the 280*e8ba3ec4SArtem Dergachev // trigger-statement (InitListExpr in this case) passed down from CFG or 281*e8ba3ec4SArtem Dergachev // otherwise always available during construction. 282*e8ba3ec4SArtem Dergachev if (dyn_cast_or_null<InitListExpr>(LCtx->getParentMap().getParent(CE))) { 283*e8ba3ec4SArtem Dergachev MemRegionManager &MRMgr = getSValBuilder().getRegionManager(); 284*e8ba3ec4SArtem Dergachev Target = MRMgr.getCXXTempObjectRegion(CE, LCtx); 285*e8ba3ec4SArtem Dergachev break; 286*e8ba3ec4SArtem Dergachev } 287*e8ba3ec4SArtem Dergachev // FALLTHROUGH 288b970505dSJordan Rose case CXXConstructExpr::CK_Delegating: { 289b970505dSJordan Rose const CXXMethodDecl *CurCtor = cast<CXXMethodDecl>(LCtx->getDecl()); 290b970505dSJordan Rose Loc ThisPtr = getSValBuilder().getCXXThis(CurCtor, 291b970505dSJordan Rose LCtx->getCurrentStackFrame()); 29205375eb4SJordan Rose SVal ThisVal = State->getSVal(ThisPtr); 293b970505dSJordan Rose 294b970505dSJordan Rose if (CE->getConstructionKind() == CXXConstructExpr::CK_Delegating) { 295b970505dSJordan Rose Target = ThisVal.getAsRegion(); 296b970505dSJordan Rose } else { 297b970505dSJordan Rose // Cast to the base type. 298599f85abSJordan Rose bool IsVirtual = 299599f85abSJordan Rose (CE->getConstructionKind() == CXXConstructExpr::CK_VirtualBase); 300599f85abSJordan Rose SVal BaseVal = getStoreManager().evalDerivedToBase(ThisVal, CE->getType(), 301599f85abSJordan Rose IsVirtual); 30220edae87SJordan Rose Target = BaseVal.getAsRegion(); 303b970505dSJordan Rose } 304b970505dSJordan Rose break; 305b970505dSJordan Rose } 306b970505dSJordan Rose } 307b970505dSJordan Rose 308fcd016e5SJordan Rose CallEventManager &CEMgr = getStateManager().getCallEventManager(); 309fcd016e5SJordan Rose CallEventRef<CXXConstructorCall> Call = 310fcd016e5SJordan Rose CEMgr.getCXXConstructorCall(CE, Target, State, LCtx); 31110e25031STed Kremenek 312f3c12ac3SJordan Rose ExplodedNodeSet DstPreVisit; 313f3c12ac3SJordan Rose getCheckerManager().runCheckersForPreStmt(DstPreVisit, Pred, CE, *this); 314b8e28654SJordan Rose 315b8e28654SJordan Rose ExplodedNodeSet PreInitialized; 316b8e28654SJordan Rose { 317b8e28654SJordan Rose StmtNodeBuilder Bldr(DstPreVisit, PreInitialized, *currBldrCtx); 318b8e28654SJordan Rose if (CE->requiresZeroInitialization()) { 319b8e28654SJordan Rose // Type of the zero doesn't matter. 320b8e28654SJordan Rose SVal ZeroVal = svalBuilder.makeZeroVal(getContext().CharTy); 321b8e28654SJordan Rose 322b8e28654SJordan Rose for (ExplodedNodeSet::iterator I = DstPreVisit.begin(), 323b8e28654SJordan Rose E = DstPreVisit.end(); 324b8e28654SJordan Rose I != E; ++I) { 325b8e28654SJordan Rose ProgramStateRef State = (*I)->getState(); 326b8e28654SJordan Rose // FIXME: Once we properly handle constructors in new-expressions, we'll 327b8e28654SJordan Rose // need to invalidate the region before setting a default value, to make 328b8e28654SJordan Rose // sure there aren't any lingering bindings around. This probably needs 329b8e28654SJordan Rose // to happen regardless of whether or not the object is zero-initialized 330b8e28654SJordan Rose // to handle random fields of a placement-initialized object picking up 331b8e28654SJordan Rose // old bindings. We might only want to do it when we need to, though. 332b8e28654SJordan Rose // FIXME: This isn't actually correct for arrays -- we need to zero- 333b8e28654SJordan Rose // initialize the entire array, not just the first element -- but our 334b8e28654SJordan Rose // handling of arrays everywhere else is weak as well, so this shouldn't 335b8e28654SJordan Rose // actually make things worse. Placement new makes this tricky as well, 336b8e28654SJordan Rose // since it's then possible to be initializing one part of a multi- 337b8e28654SJordan Rose // dimensional array. 338b570195cSAnna Zaks State = State->bindDefault(loc::MemRegionVal(Target), ZeroVal, LCtx); 3390dbb783cSCraig Topper Bldr.generateNode(CE, *I, State, /*tag=*/nullptr, 3400dbb783cSCraig Topper ProgramPoint::PreStmtKind); 341b8e28654SJordan Rose } 342b8e28654SJordan Rose } 343b8e28654SJordan Rose } 344b8e28654SJordan Rose 345afe7c2c2SJordan Rose ExplodedNodeSet DstPreCall; 346b8e28654SJordan Rose getCheckerManager().runCheckersForPreCall(DstPreCall, PreInitialized, 347fcd016e5SJordan Rose *Call, *this); 34810e25031STed Kremenek 34988bb563cSJordan Rose ExplodedNodeSet DstEvaluated; 35088bb563cSJordan Rose StmtNodeBuilder Bldr(DstPreCall, DstEvaluated, *currBldrCtx); 35188bb563cSJordan Rose 352e83cb092SJordan Rose bool IsArray = isa<ElementRegion>(Target); 353e83cb092SJordan Rose if (CE->getConstructor()->isTrivial() && 354e83cb092SJordan Rose CE->getConstructor()->isCopyOrMoveConstructor() && 355e83cb092SJordan Rose !IsArray) { 356e83cb092SJordan Rose // FIXME: Handle other kinds of trivial constructors as well. 357e83cb092SJordan Rose for (ExplodedNodeSet::iterator I = DstPreCall.begin(), E = DstPreCall.end(); 35888bb563cSJordan Rose I != E; ++I) 35988bb563cSJordan Rose performTrivialCopy(Bldr, *I, *Call); 36088bb563cSJordan Rose 36188bb563cSJordan Rose } else { 362afe7c2c2SJordan Rose for (ExplodedNodeSet::iterator I = DstPreCall.begin(), E = DstPreCall.end(); 363f3c12ac3SJordan Rose I != E; ++I) 364fcd016e5SJordan Rose defaultEvalCall(Bldr, *I, *Call); 36588bb563cSJordan Rose } 36610e25031STed Kremenek 3675b1ee2faSDevin Coughlin // If the CFG was contructed without elements for temporary destructors 3685b1ee2faSDevin Coughlin // and the just-called constructor created a temporary object then 3695b1ee2faSDevin Coughlin // stop exploration if the temporary object has a noreturn constructor. 3705b1ee2faSDevin Coughlin // This can lose coverage because the destructor, if it were present 3715b1ee2faSDevin Coughlin // in the CFG, would be called at the end of the full expression or 3725b1ee2faSDevin Coughlin // later (for life-time extended temporaries) -- but avoids infeasible 3735b1ee2faSDevin Coughlin // paths when no-return temporary destructors are used for assertions. 3745b1ee2faSDevin Coughlin const AnalysisDeclContext *ADC = LCtx->getAnalysisDeclContext(); 3755b1ee2faSDevin Coughlin if (!ADC->getCFGBuildOptions().AddTemporaryDtors) { 3765b1ee2faSDevin Coughlin const MemRegion *Target = Call->getCXXThisVal().getAsRegion(); 3775b1ee2faSDevin Coughlin if (Target && isa<CXXTempObjectRegion>(Target) && 3785b1ee2faSDevin Coughlin Call->getDecl()->getParent()->isAnyDestructorNoReturn()) { 3795b1ee2faSDevin Coughlin 3805b1ee2faSDevin Coughlin for (ExplodedNode *N : DstEvaluated) { 3815b1ee2faSDevin Coughlin Bldr.generateSink(CE, N, N->getState()); 3825b1ee2faSDevin Coughlin } 3835b1ee2faSDevin Coughlin 3845b1ee2faSDevin Coughlin // There is no need to run the PostCall and PostStmtchecker 3855b1ee2faSDevin Coughlin // callbacks because we just generated sinks on all nodes in th 3865b1ee2faSDevin Coughlin // frontier. 3875b1ee2faSDevin Coughlin return; 3885b1ee2faSDevin Coughlin } 3895b1ee2faSDevin Coughlin } 3905b1ee2faSDevin Coughlin 391afe7c2c2SJordan Rose ExplodedNodeSet DstPostCall; 39288bb563cSJordan Rose getCheckerManager().runCheckersForPostCall(DstPostCall, DstEvaluated, 393fcd016e5SJordan Rose *Call, *this); 394afe7c2c2SJordan Rose getCheckerManager().runCheckersForPostStmt(destNodes, DstPostCall, CE, *this); 39510e25031STed Kremenek } 39610e25031STed Kremenek 397b970505dSJordan Rose void ExprEngine::VisitCXXDestructor(QualType ObjectType, 39810e25031STed Kremenek const MemRegion *Dest, 39910e25031STed Kremenek const Stmt *S, 4002bc9674bSJordan Rose bool IsBaseDtor, 40110e25031STed Kremenek ExplodedNode *Pred, 40210e25031STed Kremenek ExplodedNodeSet &Dst) { 403fcd016e5SJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 404fcd016e5SJordan Rose ProgramStateRef State = Pred->getState(); 405fcd016e5SJordan Rose 40620edae87SJordan Rose // FIXME: We need to run the same destructor on every element of the array. 40720edae87SJordan Rose // This workaround will just run the first destructor (which will still 40820edae87SJordan Rose // invalidate the entire array). 4091ccc43d5SJordan Rose SVal DestVal = UnknownVal(); 4101ccc43d5SJordan Rose if (Dest) 4111ccc43d5SJordan Rose DestVal = loc::MemRegionVal(Dest); 412bc74eb1cSJordan Rose DestVal = makeZeroElementRegion(State, DestVal, ObjectType); 413bc74eb1cSJordan Rose Dest = DestVal.getAsRegion(); 41420edae87SJordan Rose 415b970505dSJordan Rose const CXXRecordDecl *RecordDecl = ObjectType->getAsCXXRecordDecl(); 416b970505dSJordan Rose assert(RecordDecl && "Only CXXRecordDecls should have destructors"); 417b970505dSJordan Rose const CXXDestructorDecl *DtorDecl = RecordDecl->getDestructor(); 418b970505dSJordan Rose 419fcd016e5SJordan Rose CallEventManager &CEMgr = getStateManager().getCallEventManager(); 420fcd016e5SJordan Rose CallEventRef<CXXDestructorCall> Call = 4212bc9674bSJordan Rose CEMgr.getCXXDestructorCall(DtorDecl, S, Dest, IsBaseDtor, State, LCtx); 422142adc49STed Kremenek 42317a8757aSJordan Rose PrettyStackTraceLoc CrashInfo(getContext().getSourceManager(), 42417a8757aSJordan Rose Call->getSourceRange().getBegin(), 42517a8757aSJordan Rose "Error evaluating destructor"); 42617a8757aSJordan Rose 4274ee71b8aSJordan Rose ExplodedNodeSet DstPreCall; 4284ee71b8aSJordan Rose getCheckerManager().runCheckersForPreCall(DstPreCall, Pred, 429fcd016e5SJordan Rose *Call, *this); 43010e25031STed Kremenek 4314ee71b8aSJordan Rose ExplodedNodeSet DstInvalidated; 432d94854a4STed Kremenek StmtNodeBuilder Bldr(DstPreCall, DstInvalidated, *currBldrCtx); 4334ee71b8aSJordan Rose for (ExplodedNodeSet::iterator I = DstPreCall.begin(), E = DstPreCall.end(); 4344ee71b8aSJordan Rose I != E; ++I) 435fcd016e5SJordan Rose defaultEvalCall(Bldr, *I, *Call); 4364ee71b8aSJordan Rose 4374ee71b8aSJordan Rose ExplodedNodeSet DstPostCall; 4384ee71b8aSJordan Rose getCheckerManager().runCheckersForPostCall(Dst, DstInvalidated, 439fcd016e5SJordan Rose *Call, *this); 44010e25031STed Kremenek } 44110e25031STed Kremenek 4428b808d64SJordan Rose void ExprEngine::VisitCXXNewAllocatorCall(const CXXNewExpr *CNE, 4438b808d64SJordan Rose ExplodedNode *Pred, 4448b808d64SJordan Rose ExplodedNodeSet &Dst) { 4458b808d64SJordan Rose ProgramStateRef State = Pred->getState(); 4468b808d64SJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 4478b808d64SJordan Rose PrettyStackTraceLoc CrashInfo(getContext().getSourceManager(), 4488b808d64SJordan Rose CNE->getStartLoc(), 4498b808d64SJordan Rose "Error evaluating New Allocator Call"); 4508b808d64SJordan Rose CallEventManager &CEMgr = getStateManager().getCallEventManager(); 4518b808d64SJordan Rose CallEventRef<CXXAllocatorCall> Call = 4528b808d64SJordan Rose CEMgr.getCXXAllocatorCall(CNE, State, LCtx); 4538b808d64SJordan Rose 4548b808d64SJordan Rose ExplodedNodeSet DstPreCall; 4558b808d64SJordan Rose getCheckerManager().runCheckersForPreCall(DstPreCall, Pred, 4568b808d64SJordan Rose *Call, *this); 4578b808d64SJordan Rose 4588b808d64SJordan Rose ExplodedNodeSet DstInvalidated; 4598b808d64SJordan Rose StmtNodeBuilder Bldr(DstPreCall, DstInvalidated, *currBldrCtx); 4608b808d64SJordan Rose for (ExplodedNodeSet::iterator I = DstPreCall.begin(), E = DstPreCall.end(); 4618b808d64SJordan Rose I != E; ++I) 4628b808d64SJordan Rose defaultEvalCall(Bldr, *I, *Call); 4638b808d64SJordan Rose getCheckerManager().runCheckersForPostCall(Dst, DstInvalidated, 4648b808d64SJordan Rose *Call, *this); 4658b808d64SJordan Rose } 4668b808d64SJordan Rose 4678b808d64SJordan Rose 46810e25031STed Kremenek void ExprEngine::VisitCXXNewExpr(const CXXNewExpr *CNE, ExplodedNode *Pred, 46910e25031STed Kremenek ExplodedNodeSet &Dst) { 470a4ee064cSJordan Rose // FIXME: Much of this should eventually migrate to CXXAllocatorCall. 471a4ee064cSJordan Rose // Also, we need to decide how allocators actually work -- they're not 472a4ee064cSJordan Rose // really part of the CXXNewExpr because they happen BEFORE the 473a4ee064cSJordan Rose // CXXConstructExpr subexpression. See PR12014 for some discussion. 47410e25031STed Kremenek 475d94854a4STed Kremenek unsigned blockCount = currBldrCtx->blockCount(); 476d519cae8STed Kremenek const LocationContext *LCtx = Pred->getLocationContext(); 4778b662704SAnton Yartsev DefinedOrUnknownSVal symVal = UnknownVal(); 4788b662704SAnton Yartsev FunctionDecl *FD = CNE->getOperatorNew(); 47910e25031STed Kremenek 4808b662704SAnton Yartsev bool IsStandardGlobalOpNewFunction = false; 4818b662704SAnton Yartsev if (FD && !isa<CXXMethodDecl>(FD) && !FD->isVariadic()) { 4828b662704SAnton Yartsev if (FD->getNumParams() == 2) { 4838b662704SAnton Yartsev QualType T = FD->getParamDecl(1)->getType(); 4848b662704SAnton Yartsev if (const IdentifierInfo *II = T.getBaseTypeIdentifier()) 4858b662704SAnton Yartsev // NoThrow placement new behaves as a standard new. 4868b662704SAnton Yartsev IsStandardGlobalOpNewFunction = II->getName().equals("nothrow_t"); 4878b662704SAnton Yartsev } 4888b662704SAnton Yartsev else 4898b662704SAnton Yartsev // Placement forms are considered non-standard. 4908b662704SAnton Yartsev IsStandardGlobalOpNewFunction = (FD->getNumParams() == 1); 4918b662704SAnton Yartsev } 4928b662704SAnton Yartsev 4938b662704SAnton Yartsev // We assume all standard global 'operator new' functions allocate memory in 4948b662704SAnton Yartsev // heap. We realize this is an approximation that might not correctly model 4958b662704SAnton Yartsev // a custom global allocator. 4968b662704SAnton Yartsev if (IsStandardGlobalOpNewFunction) 4978b662704SAnton Yartsev symVal = svalBuilder.getConjuredHeapSymbolVal(CNE, LCtx, blockCount); 4988b662704SAnton Yartsev else 4990dbb783cSCraig Topper symVal = svalBuilder.conjureSymbolVal(nullptr, CNE, LCtx, CNE->getType(), 5008b662704SAnton Yartsev blockCount); 5018b662704SAnton Yartsev 5028b662704SAnton Yartsev ProgramStateRef State = Pred->getState(); 503fcd016e5SJordan Rose CallEventManager &CEMgr = getStateManager().getCallEventManager(); 504fcd016e5SJordan Rose CallEventRef<CXXAllocatorCall> Call = 505fcd016e5SJordan Rose CEMgr.getCXXAllocatorCall(CNE, State, LCtx); 506fcd016e5SJordan Rose 507a4ee064cSJordan Rose // Invalidate placement args. 508fcd016e5SJordan Rose // FIXME: Once we figure out how we want allocators to work, 509fcd016e5SJordan Rose // we should be using the usual pre-/(default-)eval-/post-call checks here. 510fcd016e5SJordan Rose State = Call->invalidateRegions(blockCount); 511b8cb8359SJordan Rose if (!State) 512b8cb8359SJordan Rose return; 513a4ee064cSJordan Rose 514fce1b03eSPavel Labath // If this allocation function is not declared as non-throwing, failures 515fce1b03eSPavel Labath // /must/ be signalled by exceptions, and thus the return value will never be 516fce1b03eSPavel Labath // NULL. -fno-exceptions does not influence this semantics. 517fce1b03eSPavel Labath // FIXME: GCC has a -fcheck-new option, which forces it to consider the case 518fce1b03eSPavel Labath // where new can return NULL. If we end up supporting that option, we can 519fce1b03eSPavel Labath // consider adding a check for it here. 5203957fd58SJordan Rose // C++11 [basic.stc.dynamic.allocation]p3. 521fce1b03eSPavel Labath if (FD) { 5223957fd58SJordan Rose QualType Ty = FD->getType(); 5233957fd58SJordan Rose if (const FunctionProtoType *ProtoType = Ty->getAs<FunctionProtoType>()) 5243957fd58SJordan Rose if (!ProtoType->isNothrow(getContext())) 5253957fd58SJordan Rose State = State->assume(symVal, true); 5263957fd58SJordan Rose } 5273957fd58SJordan Rose 528b8cb8359SJordan Rose StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 529b8cb8359SJordan Rose 53010e25031STed Kremenek if (CNE->isArray()) { 53110e25031STed Kremenek // FIXME: allocating an array requires simulating the constructors. 53210e25031STed Kremenek // For now, just return a symbolicated region. 5336dd11048SArtem Dergachev const SubRegion *NewReg = 5346dd11048SArtem Dergachev symVal.castAs<loc::MemRegionVal>().getRegionAs<SubRegion>(); 535a4ee064cSJordan Rose QualType ObjTy = CNE->getType()->getAs<PointerType>()->getPointeeType(); 536a4ee064cSJordan Rose const ElementRegion *EleReg = 537a4ee064cSJordan Rose getStoreManager().GetElementZeroRegion(NewReg, ObjTy); 53844320367SJordan Rose State = State->BindExpr(CNE, Pred->getLocationContext(), 539632e3b7eSTed Kremenek loc::MemRegionVal(EleReg)); 54044320367SJordan Rose Bldr.generateNode(CNE, Pred, State); 54110e25031STed Kremenek return; 54210e25031STed Kremenek } 54310e25031STed Kremenek 544a4ee064cSJordan Rose // FIXME: Once we have proper support for CXXConstructExprs inside 545a4ee064cSJordan Rose // CXXNewExpr, we need to make sure that the constructed object is not 546a4ee064cSJordan Rose // immediately invalidated here. (The placement call should happen before 547a4ee064cSJordan Rose // the constructor call anyway.) 548b8cb8359SJordan Rose SVal Result = symVal; 54944320367SJordan Rose if (FD && FD->isReservedGlobalPlacementOperator()) { 55044320367SJordan Rose // Non-array placement new should always return the placement location. 55144320367SJordan Rose SVal PlacementLoc = State->getSVal(CNE->getPlacementArg(0), LCtx); 552b8cb8359SJordan Rose Result = svalBuilder.evalCast(PlacementLoc, CNE->getType(), 553aaf83184SJordan Rose CNE->getPlacementArg(0)->getType()); 55444320367SJordan Rose } 55544320367SJordan Rose 556b8cb8359SJordan Rose // Bind the address of the object, then check to see if we cached out. 557b8cb8359SJordan Rose State = State->BindExpr(CNE, LCtx, Result); 5588f6b4b04SJordan Rose ExplodedNode *NewN = Bldr.generateNode(CNE, Pred, State); 5598f6b4b04SJordan Rose if (!NewN) 5608f6b4b04SJordan Rose return; 5613503cb35SJordan Rose 5625089f3b3SJordan Rose // If the type is not a record, we won't have a CXXConstructExpr as an 5635089f3b3SJordan Rose // initializer. Copy the value over. 5645089f3b3SJordan Rose if (const Expr *Init = CNE->getInitializer()) { 5655089f3b3SJordan Rose if (!isa<CXXConstructExpr>(Init)) { 5663503cb35SJordan Rose assert(Bldr.getResults().size() == 1); 5678f6b4b04SJordan Rose Bldr.takeNodes(NewN); 568b8cb8359SJordan Rose evalBind(Dst, CNE, NewN, Result, State->getSVal(Init, LCtx), 569b8cb8359SJordan Rose /*FirstInit=*/IsStandardGlobalOpNewFunction); 5703503cb35SJordan Rose } 5713503cb35SJordan Rose } 57210e25031STed Kremenek } 57310e25031STed Kremenek 57410e25031STed Kremenek void ExprEngine::VisitCXXDeleteExpr(const CXXDeleteExpr *CDE, 57510e25031STed Kremenek ExplodedNode *Pred, ExplodedNodeSet &Dst) { 576d94854a4STed Kremenek StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 57764aea652SErik Verbruggen ProgramStateRef state = Pred->getState(); 57864aea652SErik Verbruggen Bldr.generateNode(CDE, Pred, state); 57910e25031STed Kremenek } 58010e25031STed Kremenek 5818fdb59f9STed Kremenek void ExprEngine::VisitCXXCatchStmt(const CXXCatchStmt *CS, 5828fdb59f9STed Kremenek ExplodedNode *Pred, 5838fdb59f9STed Kremenek ExplodedNodeSet &Dst) { 5848fdb59f9STed Kremenek const VarDecl *VD = CS->getExceptionDecl(); 585b1279b5cSTed Kremenek if (!VD) { 586b1279b5cSTed Kremenek Dst.Add(Pred); 587b1279b5cSTed Kremenek return; 588b1279b5cSTed Kremenek } 589b1279b5cSTed Kremenek 5908fdb59f9STed Kremenek const LocationContext *LCtx = Pred->getLocationContext(); 591d227833cSTed Kremenek SVal V = svalBuilder.conjureSymbolVal(CS, LCtx, VD->getType(), 592d94854a4STed Kremenek currBldrCtx->blockCount()); 5938fdb59f9STed Kremenek ProgramStateRef state = Pred->getState(); 594b570195cSAnna Zaks state = state->bindLoc(state->getLValue(VD, LCtx), V, LCtx); 5958fdb59f9STed Kremenek 596d94854a4STed Kremenek StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 5978fdb59f9STed Kremenek Bldr.generateNode(CS, Pred, state); 5988fdb59f9STed Kremenek } 5998fdb59f9STed Kremenek 60010e25031STed Kremenek void ExprEngine::VisitCXXThisExpr(const CXXThisExpr *TE, ExplodedNode *Pred, 60110e25031STed Kremenek ExplodedNodeSet &Dst) { 602d94854a4STed Kremenek StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 603e594034fSAnna Zaks 60410e25031STed Kremenek // Get the this object region from StoreManager. 605632e3b7eSTed Kremenek const LocationContext *LCtx = Pred->getLocationContext(); 60610e25031STed Kremenek const MemRegion *R = 60710e25031STed Kremenek svalBuilder.getRegionManager().getCXXThisRegion( 60810e25031STed Kremenek getContext().getCanonicalType(TE->getType()), 609632e3b7eSTed Kremenek LCtx); 61010e25031STed Kremenek 61149b1e38eSTed Kremenek ProgramStateRef state = Pred->getState(); 61210e25031STed Kremenek SVal V = state->getSVal(loc::MemRegionVal(R)); 613632e3b7eSTed Kremenek Bldr.generateNode(TE, Pred, state->BindExpr(TE, LCtx, V)); 61410e25031STed Kremenek } 61515843343SGabor Horvath 61615843343SGabor Horvath void ExprEngine::VisitLambdaExpr(const LambdaExpr *LE, ExplodedNode *Pred, 61715843343SGabor Horvath ExplodedNodeSet &Dst) { 61815843343SGabor Horvath const LocationContext *LocCtxt = Pred->getLocationContext(); 61915843343SGabor Horvath 62015843343SGabor Horvath // Get the region of the lambda itself. 62115843343SGabor Horvath const MemRegion *R = svalBuilder.getRegionManager().getCXXTempObjectRegion( 62215843343SGabor Horvath LE, LocCtxt); 62315843343SGabor Horvath SVal V = loc::MemRegionVal(R); 62415843343SGabor Horvath 62515843343SGabor Horvath ProgramStateRef State = Pred->getState(); 62615843343SGabor Horvath 62715843343SGabor Horvath // If we created a new MemRegion for the lambda, we should explicitly bind 62815843343SGabor Horvath // the captures. 62915843343SGabor Horvath CXXRecordDecl::field_iterator CurField = LE->getLambdaClass()->field_begin(); 63015843343SGabor Horvath for (LambdaExpr::const_capture_init_iterator i = LE->capture_init_begin(), 63115843343SGabor Horvath e = LE->capture_init_end(); 63215843343SGabor Horvath i != e; ++i, ++CurField) { 6337bdca8b2SDevin Coughlin FieldDecl *FieldForCapture = *CurField; 6347bdca8b2SDevin Coughlin SVal FieldLoc = State->getLValue(FieldForCapture, V); 6357bdca8b2SDevin Coughlin 6367bdca8b2SDevin Coughlin SVal InitVal; 6377bdca8b2SDevin Coughlin if (!FieldForCapture->hasCapturedVLAType()) { 6387bdca8b2SDevin Coughlin Expr *InitExpr = *i; 6397bdca8b2SDevin Coughlin assert(InitExpr && "Capture missing initialization expression"); 6407bdca8b2SDevin Coughlin InitVal = State->getSVal(InitExpr, LocCtxt); 6417bdca8b2SDevin Coughlin } else { 6427bdca8b2SDevin Coughlin // The field stores the length of a captured variable-length array. 6437bdca8b2SDevin Coughlin // These captures don't have initialization expressions; instead we 6447bdca8b2SDevin Coughlin // get the length from the VLAType size expression. 6457bdca8b2SDevin Coughlin Expr *SizeExpr = FieldForCapture->getCapturedVLAType()->getSizeExpr(); 6467bdca8b2SDevin Coughlin InitVal = State->getSVal(SizeExpr, LocCtxt); 6477bdca8b2SDevin Coughlin } 6487bdca8b2SDevin Coughlin 649b570195cSAnna Zaks State = State->bindLoc(FieldLoc, InitVal, LocCtxt); 65015843343SGabor Horvath } 65115843343SGabor Horvath 65215843343SGabor Horvath // Decay the Loc into an RValue, because there might be a 65315843343SGabor Horvath // MaterializeTemporaryExpr node above this one which expects the bound value 65415843343SGabor Horvath // to be an RValue. 65515843343SGabor Horvath SVal LambdaRVal = State->getSVal(R); 65615843343SGabor Horvath 65715843343SGabor Horvath ExplodedNodeSet Tmp; 65815843343SGabor Horvath StmtNodeBuilder Bldr(Pred, Tmp, *currBldrCtx); 65915843343SGabor Horvath // FIXME: is this the right program point kind? 66015843343SGabor Horvath Bldr.generateNode(LE, Pred, 66115843343SGabor Horvath State->BindExpr(LE, LocCtxt, LambdaRVal), 66215843343SGabor Horvath nullptr, ProgramPoint::PostLValueKind); 66315843343SGabor Horvath 66415843343SGabor Horvath // FIXME: Move all post/pre visits to ::Visit(). 66515843343SGabor Horvath getCheckerManager().runCheckersForPostStmt(Dst, Tmp, LE, *this); 66615843343SGabor Horvath } 667