On Tue, Oct 24, 2017 at 10:47:32PM +0100, Michael Collison wrote:
> James,
>
> The patch was test as required. However when I tested with the latest trunk
> there were two test failures that need to be updated because of my patch.
>
> The files gcc.target/aarch64/vect-vcvt.c was failing because
7 9:43 AM
To: Michael Collison
Cc: GCC Patches ; nd
Subject: Re: [PING][PATCH][Aarch64] Improve int<->FP conversions
On Tue, Oct 17, 2017 at 01:17:04AM +0100, Michael Collison wrote:
> Patch updated with all comments from James.
OK with an appropriate ChangeLog and assuming it has be
On Tue, Oct 17, 2017 at 01:17:04AM +0100, Michael Collison wrote:
> Patch updated with all comments from James.
OK with an appropriate ChangeLog and assuming it has been tested as
required.
Thanks,
James
Reviewed-by:
lgh [mailto:james.greenha...@arm.com]
Sent: Wednesday, October 4, 2017 2:39 AM
To: Michael Collison
Cc: GCC Patches ; nd
Subject: Re: [PING][PATCH][Aarch64] Improve int<->FP conversions
On Sun, Oct 01, 2017 at 02:07:57AM +0100, Michael Collison wrote:
> Sorry. Here is the patch.
I think this n
On Sun, Oct 01, 2017 at 02:07:57AM +0100, Michael Collison wrote:
> Sorry. Here is the patch.
I think this needs a small amount fo rework in iterators.md - the names
you've used don't follow conventions in that file (e.g. "V" normally has
something to do with vectors) so could do with patching up.
Sorry. Here is the patch.
-Original Message-
From: James Greenhalgh [mailto:james.greenha...@arm.com]
Sent: Friday, September 29, 2017 3:57 PM
To: Michael Collison
Cc: GCC Patches ; nd
Subject: Re: [PING][PATCH][Aarch64] Improve int<->FP conversions
On Fri, Sep 29, 2017 at 10:4
On Fri, Sep 29, 2017 at 10:42:14PM +0100, Michael Collison wrote:
> Ping. Original patch posted here:
>
> https://gcc.gnu.org/ml/gcc-patches/2017-09/msg00808.html
I don't see a patch attached there to review, nor can I see an earlier mail
threaded to it with the patch attached. Can you help?
Tha
Ping. Original patch posted here:
https://gcc.gnu.org/ml/gcc-patches/2017-09/msg00808.html
Resending this patch as I did not have the correct subject line.
This patch improves the latency of code by eliminating two FP <-> integer
register transfers.
An example:
float f1(float x)
{
int y = x;
return (float)y;
}
Trunk generates:
f1:
fcvtzs w0, s0
scvtf s0, w0