Fixed bug where GRConstants::AddBindings() did not check for values

tkremenek · tkremenek · commit 7e7070b59441 · 2008-01-16T19:28:16.000Z
that were "not a constant."

llvm-svn: 46085
diff --git a/clang/Analysis/GRConstants.cpp b/clang/Analysis/GRConstants.cpp
@@ -225,7 +225,8 @@ ExprVariantTy GRConstants::GetBinding(Expr* E) {
 }
 
 void GRConstants::AddBinding(Expr* E, ExprVariantTy V, bool isBlkLvl) {
-  CurrentState = StateMgr.Add(CurrentState, DSPtr(E,isBlkLvl), V.getVal());
+  if (V) 
+    CurrentState = StateMgr.Add(CurrentState, DSPtr(E,isBlkLvl), V.getVal());
 }
 
 void GRConstants::SwitchNodeSets() {

Original file line number	Diff line number	Diff line change
`@@ -225,7 +225,8 @@ ExprVariantTy GRConstants::GetBinding(Expr* E) {`
`225`	`225`	`}`
`226`	`226`
`227`	`227`	`void GRConstants::AddBinding(Expr* E, ExprVariantTy V, bool isBlkLvl) {`
`228`		`- CurrentState = StateMgr.Add(CurrentState, DSPtr(E,isBlkLvl), V.getVal());`
	`228`	`+ if (V)`
	`229`	`+ CurrentState = StateMgr.Add(CurrentState, DSPtr(E,isBlkLvl), V.getVal());`
`229`	`230`	`}`
`230`	`231`
`231`	`232`	`void GRConstants::SwitchNodeSets() {`