[cfe-commits] r108004 - /cfe/trunk/lib/Sema/SemaChecking.cpp

Chandler Carruth chandlerc at gmail.com
Fri Jul 9 12:19:40 PDT 2010

Author: chandlerc
Date: Fri Jul  9 14:19:40 2010
New Revision: 108004

URL: http://llvm.org/viewvc/llvm-project?rev=108004&view=rev
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.


Modified: cfe/trunk/lib/Sema/SemaChecking.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaChecking.cpp?rev=108004&r1=108003&r2=108004&view=diff
--- cfe/trunk/lib/Sema/SemaChecking.cpp (original)
+++ cfe/trunk/lib/Sema/SemaChecking.cpp Fri Jul  9 14:19:40 2010
@@ -570,13 +570,12 @@
     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;
       llvm_unreachable("Unhandled original value type!");

More information about the cfe-commits mailing list