[llvm-commits] [llvm] r65601 - /llvm/trunk/lib/Target/PowerPC/PPCSubtarget.h

[Dale Johannesen]

Author: johannes
Date: Thu Feb 26 18:56:35 2009
New Revision: 65601

URL: http://llvm.org/viewvc/llvm-project?rev=65601&view=rev
Log:
Alignment values for i64 and f64 on ppc64 were wrong,
possibly for the reason suggested by the comment.
No wonder it didn't work very well.  This unblocks
bootstrap with assertions on ppc.


Modified:
    llvm/trunk/lib/Target/PowerPC/PPCSubtarget.h

Modified: llvm/trunk/lib/Target/PowerPC/PPCSubtarget.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PowerPC/PPCSubtarget.h?rev=65601&r1=65600&r2=65601&view=diff

==============================================================================
--- llvm/trunk/lib/Target/PowerPC/PPCSubtarget.h (original)
+++ llvm/trunk/lib/Target/PowerPC/PPCSubtarget.h Thu Feb 26 18:56:35 2009
@@ -108,7 +108,9 @@
   /// getTargetDataString - Return the pointer size and type alignment
   /// properties of this subtarget.
   const char *getTargetDataString() const {
-    return isPPC64() ? "E-p:64:64-f64:32:64-i64:32:64-f128:64:128"
+    // Note, the alignment values for f64 and i64 on ppc64 in Darwin
+    // documentation are wrong; these are correct (i.e. "what gcc does").
+    return isPPC64() ? "E-p:64:64-f64:64:64-i64:64:64-f128:64:128"
                      : "E-p:32:32-f64:32:64-i64:32:64-f128:64:128";
   }