This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][GCC][AArch64] optimize float immediate moves (2 /4) - HF/DF/SF mode.
- From: "Bin.Cheng" <amker dot cheng at gmail dot com>
- To: Tamar Christina <Tamar dot Christina at arm dot com>
- Cc: James Greenhalgh <James dot Greenhalgh at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, nd <nd at arm dot com>, Marcus Shawcroft <Marcus dot Shawcroft at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Tue, 1 Aug 2017 13:04:12 +0100
- Subject: Re: [PATCH][GCC][AArch64] optimize float immediate moves (2 /4) - HF/DF/SF mode.
- Authentication-results: sourceware.org; auth=none
- References: <VI1PR0801MB203117D353E8A26AA9909037FFC80@VI1PR0801MB2031.eurprd08.prod.outlook.com> <20170614084233.GA15599@arm.com> <VI1PR0801MB2031791FA5E8C0CF053CB699FFDA0@VI1PR0801MB2031.eurprd08.prod.outlook.com> <VI1PR0801MB20314F94AA32F6209F57D06CFFDF0@VI1PR0801MB2031.eurprd08.prod.outlook.com> <CAHFci28Kxt=VeFSZHd896eAbp+mJMiN7fZcbDe_P8CCOfNeVfQ@mail.gmail.com> <HE1PR0802MB23166ED79F8AA86E19157102FFB30@HE1PR0802MB2316.eurprd08.prod.outlook.com>
On Tue, Aug 1, 2017 at 12:51 PM, Tamar Christina
<Tamar.Christina@arm.com> wrote:
>>
>> Given review comment already pointed out big-endian issue and patch was
>> updated to address it, I would expect reg-test on a big-endian target before
>> applying patch, right?
>
> The patch spent 6 months in external review.
> Given that, I simply forgot to rerun big endian before the commit as I did the rest.
>
> The failing tests were all added after the submission of this patch. I'll have a look.
I may bisect to wrong commit? The patch is committed three days ago,
how could all failing tests be added after it?
Thanks,
bin