[PATCH v3] [PPC64]: Add support for Swift calling convention

2017-06-21 Thread Andrew Jeffery via cfe-commits
For the tests I've extracted the int5 and int8 cases to cater for different alignments for different platform ABIs. For Linux on POWER the 5 and 8 element vectors must be naturally aligned with respect to the total "soft" vector size, despite being represented as an aggregate. Specifically, the pat

[PATCH] [PPC64]: Add support for Swift calling convention

2017-03-22 Thread Andrew Jeffery via cfe-commits
Signed-off-by: Andrew Jeffery --- lib/Basic/Targets.cpp | 11 +++ lib/CodeGen/TargetInfo.cpp | 14 -- 2 files changed, 23 insertions(+), 2 deletions(-) diff --git a/lib/Basic/Targets.cpp b/lib/Basic/Targets.cpp index fdedd91a3e15..6d7f37f2479a 100644 --- a/lib/Basic/Targ

Re: [PATCH] [PPC64]: Add support for Swift calling convention

2017-04-05 Thread Andrew Jeffery via cfe-commits
Ping - Is anyone happy to review or apply the patch? If I've missed some process I should have followed, please let me know! Andrew On Wed, 2017-03-22 at 22:30 +1030, Andrew Jeffery wrote: > Signed-off-by: Andrew Jeffery > --- >  lib/Basic/Targets.cpp  | 11 +++ >  lib/CodeGen/Target

Re: [PATCH] [PPC64]: Add support for Swift calling convention

2017-04-12 Thread Andrew Jeffery via cfe-commits
On Wed, 2017-04-12 at 14:01 -0700, Adrian Prantl wrote: > Is it possible to add a testcase for this? I will look into it. Thanks for the review! Andrew > > -- adrian > > > > On Mar 22, 2017, at 5:00 AM, Andrew Jeffery via cfe-commits > > wrote: > > >

Re: [PATCH v3] [PPC64]: Add support for Swift calling convention

2017-07-05 Thread Andrew Jeffery via cfe-commits
Ping - is anyone able to provide feedback? Cheers, Andrew On Thu, 2017-06-22 at 16:02 +0930, Andrew Jeffery wrote: > For the tests I've extracted the int5 and int8 cases to cater for > different alignments for different platform ABIs. For Linux on POWER the > 5 and 8 element vectors must be natu

Re: [PATCH v3] [PPC64]: Add support for Swift calling convention

2017-07-18 Thread Andrew Jeffery via cfe-commits
Another friendly ping :) Cheers, Andrew On Wed, 2017-07-05 at 11:23 +0930, Andrew Jeffery wrote: > Ping - is anyone able to provide feedback? > > Cheers, > > Andrew > > On Thu, 2017-06-22 at 16:02 +0930, Andrew Jeffery wrote: > > For the tests I've extracted the int5 and int8 cases to cater f

Re: [PATCH v3] [PPC64]: Add support for Swift calling convention

2017-07-23 Thread Andrew Jeffery via cfe-commits
On Wed, 2017-07-19 at 08:26 -0700, Adrian Prantl wrote: > > > > On Jun 21, 2017, at 11:32 PM, Andrew Jeffery wrote: > > > > For the tests I've extracted the int5 and int8 cases to cater for > > different alignments for different platform ABIs. For Linux on POWER the > > 5 and 8 element vectors mu

Re: [PATCH v3] [PPC64]: Add support for Swift calling convention

2017-07-23 Thread Andrew Jeffery via cfe-commits
Hi Hal, On Sat, 2017-07-22 at 23:26 -0500, Hal Finkel wrote: > On 07/19/2017 10:26 AM, Adrian Prantl wrote: > > > > > > On Jun 21, 2017, at 11:32 PM, Andrew Jeffery > > > > > > wrote: > > > > > > For the tests I've extracted the int5 and int8 cases to cater for > > > different alignments for di

Re: [PATCH v3] [PPC64]: Add support for Swift calling convention

2017-07-24 Thread Andrew Jeffery via cfe-commits
On Mon, 2017-07-24 at 09:32 +0200, Ulrich Weigand wrote: > > Andrew Jeffery wrote on 24.07.2017 03:54:05: > > > > > > +  bool shouldPassIndirectlyForSwift(CharUnits totalSize, > > > > > +ArrayRef scalars, > > > > > +bool asRe

[PATCH v2] [PPC64]: Add support for Swift calling convention

2017-04-26 Thread Andrew Jeffery via cfe-commits
For the tests I've extracted the int5 and int8 cases to cater for different alignments for different platform ABIs. For Linux on POWER the 5 and 8 element vectors must be naturally aligned with respect to the total "soft" vector size, despite being represented as an aggregate. Specifically, the pat

Re: [PATCH v2] [PPC64]: Add support for Swift calling convention

2017-05-11 Thread Andrew Jeffery via cfe-commits
Ping - does anyone have a moment to take a look? Cheers, Andrew On Thu, 2017-04-27 at 13:52 +0930, Andrew Jeffery wrote: > For the tests I've extracted the int5 and int8 cases to cater for > different alignments for different platform ABIs. For Linux on POWER the > 5 and 8 element vectors must b