[llvm] r197524 - Handle i64 first for clarity. No functionality change.

Rafael Espindola rafael.espindola at gmail.com
Tue Dec 17 13:28:36 PST 2013


Author: rafael
Date: Tue Dec 17 15:28:36 2013
New Revision: 197524

URL: http://llvm.org/viewvc/llvm-project?rev=197524&view=rev
Log:
Handle i64 first for clarity. No functionality change.

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=197524&r1=197523&r2=197524&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMTargetMachine.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMTargetMachine.cpp Tue Dec 17 15:28:36 2013
@@ -76,13 +76,14 @@ static std::string computeDataLayout(ARM
   if (ST.isThumb())
     Ret += "-i1:8:32-i8:8:32-i16:16: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.
+  // ABIs other than APC have 64 bit integers with natural alignment.
+  if (!ST.isAPCS_ABI())
+    Ret += "-i64:64";
+
+  // We have 64 bits floats. The APCS ABI requires them to be aligned to 32
+  // bits, others to 64 bits. We always try to align to 64 bits.
   if (ST.isAPCS_ABI())
     Ret += "-f64:32:64";
-  else
-    Ret += "-i64:64";
 
   // 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.





More information about the llvm-commits mailing list