[cfe-commits] r60206 - /cfe/trunk/lib/Analysis/GRExprEngine.cpp

[Zhongxing Xu]

Author: zhongxingxu
Date: Fri Nov 28 02:34:30 2008
New Revision: 60206

URL: http://llvm.org/viewvc/llvm-project?rev=60206&view=rev
Log:
Code cleanup. No functional change.

Modified:
    cfe/trunk/lib/Analysis/GRExprEngine.cpp

Modified: cfe/trunk/lib/Analysis/GRExprEngine.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Analysis/GRExprEngine.cpp?rev=60206&r1=60205&r2=60206&view=diff

==============================================================================
--- cfe/trunk/lib/Analysis/GRExprEngine.cpp (original)
+++ cfe/trunk/lib/Analysis/GRExprEngine.cpp Fri Nov 28 02:34:30 2008
@@ -982,6 +982,7 @@
 
   // FIXME: The "CheckOnly" option exists only because Array and Field
   //  loads aren't fully implemented.  Eventually this option will go away.
+  assert(!CheckOnly);
 
   if (CheckOnly)
     MakeNode(Dst, Ex, Pred, St, K);
@@ -989,9 +990,10 @@
     // This is important.  We must nuke the old binding.
     MakeNode(Dst, Ex, Pred, BindExpr(St, Ex, UnknownVal()), K);
   }
-  else    
-    MakeNode(Dst, Ex, Pred, BindExpr(St, Ex, GetSVal(St, cast<Loc>(location),
-                                                     Ex->getType())), K);  
+  else {
+    SVal V = GetSVal(St, cast<Loc>(location), Ex->getType());
+    MakeNode(Dst, Ex, Pred, BindExpr(St, Ex, V), K);  
+  }
 }
 
 void GRExprEngine::EvalStore(NodeSet& Dst, Expr* Ex, Expr* StoreE, NodeTy* Pred,