Rip out the floating point return type handling from the atomic builtin. It's

wrong, and we don't handle floating point value type arguments yet anyways.
Will add correct logic for both when I finish the patch.

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

diff --git a/lib/Sema/SemaChecking.cpp b/lib/Sema/SemaChecking.cpp
index 6a818bd5757e39ddfd3a652f822fbe2ec922d4a4..217ada01c6b8233b91fabdcc60f5835dc62b981e 100644 (file)
--- a/lib/Sema/SemaChecking.cpp
+++ b/lib/Sema/SemaChecking.cpp
@@ -570,13 +570,12 @@ Sema::SemaBuiltinAtomicOverloaded(OwningExprResult TheCallResult) {
 
     assert(ValType->isIntegerType() &&
            "We always convert atomic operation values to integers.");
+    // FIXME: Handle floating point value type here too.
     CastExpr::CastKind Kind;
     if (OrigValType->isIntegerType())
       Kind = CastExpr::CK_IntegralCast;
     else if (OrigValType->hasPointerRepresentation())
       Kind = CastExpr::CK_IntegralToPointer;
-    else if (OrigValType->isRealFloatingType())
-      Kind = CastExpr::CK_IntegralToFloating;
     else
       llvm_unreachable("Unhandled original value type!");