This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][compare-elim] Merge zero-comparisons with normal ops
- From: Jeff Law <law at redhat dot com>
- To: Michael Collison <Michael dot Collison at arm dot com>, Segher Boessenkool <segher at kernel dot crashing dot org>, Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, nd <nd at arm dot com>
- Date: Fri, 13 Oct 2017 12:04:16 -0600
- Subject: Re: [PATCH][compare-elim] Merge zero-comparisons with normal ops
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx02.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx02.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=law at redhat dot com
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com CFE01883AC
- References: <HE1PR0802MB2377CC3CF0D995F4BC869D2595880@HE1PR0802MB2377.eurprd08.prod.outlook.com> <99ac3801-9894-fe63-b712-2bcfd637a859@redhat.com> <59A5282A.7060705@foss.arm.com> <20170901230726.GS13471@gate.crashing.org> <HE1PR0802MB2377818A883A7B663207D6B395970@HE1PR0802MB2377.eurprd08.prod.outlook.com>
On 09/06/2017 09:56 AM, Michael Collison wrote:
> Patch updated with all relevant comments and suggestions.
>
> Bootstrapped and tested on arm-none-linux-gnueabihf, and aarch64-none-linux-gnu and x86_64.
>
> Ok for trunk?
>
> 2017-08-05 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
> Michael Collison <michael.collison@arm.com>
>
> * compare-elim.c: Include emit-rtl.h.
> (can_merge_compare_into_arith): New function.
> (try_validate_parallel): Likewise.
> (try_merge_compare): Likewise.
> (try_eliminate_compare): Call the above when no previous clobber
> is available.
> (execute_compare_elim_after_reload): Add DF_UD_CHAIN and DF_DU_CHAIN
> dataflow problems.
>
> 2017-08-05 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
> Michael Collison <michael.collison@arm.com>
>
> * gcc.target/aarch64/cmpelim_mult_uses_1.c: New test.
Sorry for the delay.
This looks good. OK for the trunk.
jeff