From: Zhanyong Wan <wan@google.com>
Date: Wed, 24 Nov 2010 01:47:11 +0000 (+0000)
Subject: Add an assert() to catch errors using EvalLoad().  Reviewed by kremenek.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cca8ab155e8c20b98ba2d90eb2b1c228895e06fd;p=clang

Add an assert() to catch errors using EvalLoad().  Reviewed by kremenek.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@120073 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Checker/GRExprEngine.cpp b/lib/Checker/GRExprEngine.cpp
index 4483cdb76c..30ac9cefbb 100644
--- a/lib/Checker/GRExprEngine.cpp
+++ b/lib/Checker/GRExprEngine.cpp
@@ -1957,10 +1957,11 @@ void GRExprEngine::EvalStore(ExplodedNodeSet& Dst, const Expr *AssignE,
     EvalBind(Dst, StoreE, *NI, GetState(*NI), location, Val);
 }
 
-void GRExprEngine::EvalLoad(ExplodedNodeSet& Dst, const Expr *Ex, 
+void GRExprEngine::EvalLoad(ExplodedNodeSet& Dst, const Expr *Ex,
                             ExplodedNode* Pred,
                             const GRState* state, SVal location,
                             const void *tag, QualType LoadTy) {
+  assert(!isa<NonLoc>(location) && "location cannot be a NonLoc.");
 
   // Are we loading from a region?  This actually results in two loads; one
   // to fetch the address of the referenced value and one to fetch the