This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][RS6000] Migrate from reduc_xxx to reduc_xxx_scal optabs
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>, Alan Lawrence <alan dot lawrence at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 18 Sep 2015 14:31:48 -0400
- Subject: Re: [PATCH][RS6000] Migrate from reduc_xxx to reduc_xxx_scal optabs
- Authentication-results: sourceware.org; auth=none
- References: <1442590748-5801-1-git-send-email-alan dot lawrence at arm dot com> <1442600764 dot 10907 dot 39 dot camel at gnopaine>
On Fri, Sep 18, 2015 at 2:26 PM, Bill Schmidt
<wschmidt@linux.vnet.ibm.com> wrote:
> On Fri, 2015-09-18 at 16:39 +0100, Alan Lawrence wrote:
>> This is a respin of https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01024.html
>> after discovering that patch was broken on power64le - thanks to Bill Schmidt
>> for pointing out that gcc112 is the opposite endianness to gcc110...
>>
>> This time I decided to avoid any funny business with making RTL match other
>> patterns in other .md files, and instead to directly call the relevant
>> expanders. This should thus preserve the codegen of the previous expansion path.
>> Moreover, combining the uplus and splus expansion paths (as addition is the same
>> regardless of signedness) causes some additional examples to be reduced directly
>> via patterns.
>
> Alan, thanks for the patch! David will have to approve it, but this
> endian-corrected version looks good to me.
Okay with me.
Thanks, David