[PATCH] D15120: Add support for __float128 type to be used by targets that support it

John McCall via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 7 00:16:36 PST 2015


rjmccall added a reviewer: akyrtzi.

================
Comment at: lib/AST/ASTContext.cpp:5424
@@ -5410,1 +5423,3 @@
+    // FIXME: need to figure out what this is for __float128
+    case BuiltinType::Float128:   return 'K';
     case BuiltinType::NullPtr:    return '*'; // like char*
----------------
Please add this as a case to the block below for Half.  I'll ping internally for a decision on what @encoding to use.

================
Comment at: lib/Index/USRGeneration.cpp:604
@@ -602,1 +603,3 @@
+        case BuiltinType::Float128:
+          c = ' '; break;
         case BuiltinType::NullPtr:
----------------
I'm pretty sure you can just choose something here that doesn't collide, but the right person to ask is Argyrios.


Repository:
  rL LLVM

http://reviews.llvm.org/D15120





More information about the cfe-commits mailing list