[llvm] r196850 - Add comments documenting the ARM datalayout string.

[Rafael Espindola]

Author: rafael
Date: Mon Dec  9 18:37:37 2013
New Revision: 196850

URL: http://llvm.org/viewvc/llvm-project?rev=196850&view=rev
Log:
Add comments documenting the ARM datalayout string.

Modified:
    llvm/trunk/lib/Target/ARM/ARMTargetMachine.cpp

Modified: llvm/trunk/lib/Target/ARM/ARMTargetMachine.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMTargetMachine.cpp?rev=196850&r1=196849&r2=196850&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMTargetMachine.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMTargetMachine.cpp Mon Dec  9 18:37:37 2013
@@ -68,26 +68,38 @@ void ARMBaseTargetMachine::addAnalysisPa
 void ARMTargetMachine::anchor() { }
 
 static std::string computeDataLayout(ARMSubtarget &ST) {
+  // Little endian. Pointers are 32 bits and aligned to 32 bits.
   std::string Ret = "e-p:32:32";
 
+  // We have 64 bits floats and integers. The APCS ABI requires them to be
+  // aligned s them to 32 bits, others to 64 bits. We always try to align to
+  // 64 bits.
   if (ST.isAPCS_ABI())
     Ret += "-f64:32:64-i64:32:64";
   else
     Ret += "-f64:64:64-i64:64:64";
 
+  // On thumb, i16,i18 and i1 have natural aligment requirements, but we try to
+  // align to 32.
   if (ST.isThumb())
     Ret += "-i16:16:32-i8:8:32-i1:8:32";
 
+  // We have 128 and 64 bit vectors. The APCS ABI aligns them to 32 bits, others
+  // to 64. We always ty to give them natural alignment.
   if (ST.isAPCS_ABI())
     Ret += "-v128:32:128-v64:32:64";
   else
     Ret += "-v128:64:128-v64:64:64";
 
+  // An aggregate of size 0 is ABI aligned to 0.
+  // FIXME: explain better what this means.
   if (ST.isThumb())
     Ret += "-a:0:32";
 
+  // Integer registers are 32 bits.
   Ret += "-n32";
 
+  // The stack is 64 bit aligned on AAPCS and 32 bit aligned everywhere else.
   if (ST.isAAPCS_ABI())
     Ret += "-S64";
   else