Explicitly check for casts to double or complex types instead of possibly asserting...

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

diff --git a/lib/Checker/SValuator.cpp b/lib/Checker/SValuator.cpp
index 66cd3193b14dd608b3b4647613b04c3c213e3816..fd2bbd06fb43c74632f0f91629c0f80dd7eca87c 100644 (file)
--- a/lib/Checker/SValuator.cpp
+++ b/lib/Checker/SValuator.cpp
@@ -66,6 +66,12 @@ SValuator::CastResult SValuator::EvalCast(SVal val, const GRState *state,
     if (C.hasSameUnqualifiedType(castTy, originalTy))
       return CastResult(state, val);
 
+  // Check for casts to real or complex numbers.  We don't handle these at all
+  // right now.
+  if (castTy->isFloatingType() || castTy->isAnyComplexType())
+    return CastResult(state, UnknownVal());
+  
+  // Check for casts from integers to integers.
   if (castTy->isIntegerType() && originalTy->isIntegerType())
     return CastResult(state, EvalCastNL(cast<NonLoc>(val), castTy));
 

diff --git a/test/Analysis/misc-ps.m b/test/Analysis/misc-ps.m
index acf49f8e6773b49e15ca7555644c3952ce3dba3c..b31cd4cdd195cb2b0849f9895f9b7ccc852cf954 100644 (file)
--- a/test/Analysis/misc-ps.m
+++ b/test/Analysis/misc-ps.m
@@ -851,3 +851,18 @@ int rdar_7593875(int n) {
   // Previously we got a false positive about 'v' being uninitialized.
   return v; // no-warning
 }
+
+//===----------------------------------------------------------------------===//
+// Handle casts from symbolic regions (packaged as integers) to doubles.
+// Previously this caused an assertion failure.
+//===----------------------------------------------------------------------===//
+
+void *foo_rev95119();
+void baz_rev95119(double x);
+void bar_rev95119() {
+  // foo_rev95119() returns a symbolic pointer.  It is then 
+  // cast to an int which is then cast to a double.
+  int value = (int) foo_rev95119();
+  baz_rev95119((double)value);
+}
+