Check some cases for previous patch. Make the code more robust.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@57317 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Analysis/GRExprEngine.cpp b/lib/Analysis/GRExprEngine.cpp
index a6b5013..c391c6f 100644
--- a/lib/Analysis/GRExprEngine.cpp
+++ b/lib/Analysis/GRExprEngine.cpp
@@ -1132,10 +1132,14 @@
case 5:
if (!memcmp(s, "panic", 5)) Builder->BuildSinks = true;
else if (!memcmp(s, "error", 5)) {
- Expr* Arg = *CE->arg_begin();
- if (IntegerLiteral* IL = dyn_cast<IntegerLiteral>(Arg))
- if (IL->getValue() != 0)
+ if (CE->getNumArgs() > 0) {
+ RVal X = GetRVal(St, *CE->arg_begin());
+ // FIXME: use Assume to inspect the possible symbolic value of
+ // X. Also check the specific signature of error().
+ nonlval::ConcreteInt* CI = dyn_cast<nonlval::ConcreteInt>(&X);
+ if (CI && CI->getValue() != 0)
Builder->BuildSinks = true;
+ }
}
break;