[llvm-commits] [llvm] r65661 - /llvm/branches/Apple/Dib/lib/Target/PowerPC/PPCSubtarget.h

[Bill Wendling]

Author: void
Date: Fri Feb 27 17:54:28 2009
New Revision: 65661

URL: http://llvm.org/viewvc/llvm-project?rev=65661&view=rev
Log:
--- Merging (from foreign repository) r65601 into '.':
U    lib/Target/PowerPC/PPCSubtarget.h

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/branches/Apple/Dib/lib/Target/PowerPC/PPCSubtarget.h

Modified: llvm/branches/Apple/Dib/lib/Target/PowerPC/PPCSubtarget.h
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/Apple/Dib/lib/Target/PowerPC/PPCSubtarget.h?rev=65661&r1=65660&r2=65661&view=diff

==============================================================================
--- llvm/branches/Apple/Dib/lib/Target/PowerPC/PPCSubtarget.h (original)
+++ llvm/branches/Apple/Dib/lib/Target/PowerPC/PPCSubtarget.h Fri Feb 27 17:54:28 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";
   }