[cfe-commits] r70465 - /cfe/trunk/lib/AST/ASTContext.cpp

Chris Lattner sabre at nondot.org
Wed Apr 29 19:55:14 PDT 2009


Author: lattner
Date: Wed Apr 29 21:55:13 2009
New Revision: 70465

URL: http://llvm.org/viewvc/llvm-project?rev=70465&view=rev
Log:
int128_t is apparently 128-bit aligned on all 64-bit targets, and
not supported on 32-bit targets, so we can define it to be 128-bit
aligned there too :)

Modified:
    cfe/trunk/lib/AST/ASTContext.cpp

Modified: cfe/trunk/lib/AST/ASTContext.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ASTContext.cpp?rev=70465&r1=70464&r2=70465&view=diff

==============================================================================
--- cfe/trunk/lib/AST/ASTContext.cpp (original)
+++ cfe/trunk/lib/AST/ASTContext.cpp Wed Apr 29 21:55:13 2009
@@ -413,6 +413,11 @@
       Width = Target.getLongLongWidth();
       Align = Target.getLongLongAlign();
       break;
+    case BuiltinType::Int128:
+    case BuiltinType::UInt128:
+      Width = 128;
+      Align = 128; // int128_t is 128-bit aligned on all targets.
+      break;
     case BuiltinType::Float:
       Width = Target.getFloatWidth();
       Align = Target.getFloatAlign();
@@ -425,13 +430,6 @@
       Width = Target.getLongDoubleWidth();
       Align = Target.getLongDoubleAlign();
       break;
-    case BuiltinType::Int128:
-    case BuiltinType::UInt128:
-      Width = 128;
-          
-      // FIXME: Is this correct for all targets?
-      Align = 128;
-      break;
     }
     break;
   case Type::FixedWidthInt:





More information about the cfe-commits mailing list