[PATCH] AAPCS: Cannot split argument between GPRs and the stack after a CPRC has been allocated to the stack

Oliver Stannard oliver.stannard at arm.com
Mon Feb 10 03:10:06 PST 2014



================
Comment at: lib/CodeGen/TargetInfo.cpp:3670
@@ -3597,2 +3669,3 @@
     SizeRegs = (getContext().getTypeSize(Ty) + 63) / 64;
+    markAllocatedGPRs(AllocatedGPRs, 2, SizeRegs * 2);
   }
----------------
Renato Golin wrote:
> Why SizeRegs * 2? Wouldn't they be 8-byte aligned anyway?
SizeRegs is in units of "number of 64-bit registers" (actually implemented as aligned pairs of GPRs), but markAllocatedGPRs expects the size in units of "number of 32-bit registers".

================
Comment at: lib/CodeGen/TargetInfo.cpp:3481
@@ -3457,1 +3480,3 @@
+    // the stack.
+    AllocatedVFP = 17;
     return;
----------------
Renato Golin wrote:
> Why are you saturating? Is there any place that especially check against 17? Or are all of them like this, >=16?
On line 3336, we check whether AllocatedVFP > 16, as we need to know whether some CPRCs have been allocated on the stack.


http://llvm-reviews.chandlerc.com/D2726



More information about the cfe-commits mailing list