[PATCH] D101010: [SystemZ] [z/OS] Add XPLINK64 Calling Convention to SystemZ

[Kevin P. Neal via Phabricator via llvm-commits]

kpn added inline comments.


================
Comment at: llvm/lib/Target/SystemZ/SystemZCallingConv.td:278
+  // The first 4 named  float and double arguments are passed in registers FPR0-FPR6.
+  // The rest will be passed in the user area.
+  CCIfType<[f32, f64], CCIfFixed<CCCustom<"CC_XPLINK64_Shadow_Reg">>>,
----------------
Everybody0523 wrote:
> kpn wrote:
> > Don't you need to hedge a bit in this comment? Some FP arguments aren't passed in registers if there are too many integer arguments. (Unless I'm wrong and the restriction only applies to 32-bit XPLINK?)
> If I am not mistaken you are referring to the requirement that a FP argument is within 15 words of the previous FP argument in the argument list?
> 
> If so, then yes - XPLINK64 has no such clause. 
That's the one, yes. It must be a 31-bit only restriction then. Noted. Thanks for the clarification!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101010/new/

https://reviews.llvm.org/D101010