This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PING][PATCH] Move the check for any_condjump_p from sched-deps to target macros
- From: Jeff Law <law at redhat dot com>
- To: "Hurugalawadi, Naveen" <Naveen dot Hurugalawadi at cavium dot com>, Wilco Dijkstra <Wilco dot Dijkstra at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: "Pinski, Andrew" <Andrew dot Pinski at cavium dot com>, "roland dot illig at gmx dot de" <roland dot illig at gmx dot de>, "joseph at codesourcery dot com" <joseph at codesourcery dot com>, "dmalcolm at redhat dot com" <dmalcolm at redhat dot com>, nd <nd at arm dot com>, James Greenhalgh <James dot Greenhalgh at arm dot com>, Marcus Shawcroft <marcus dot shawcroft at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, "ramana dot radhakrishnan at foss dot arm dot com" <ramana dot radhakrishnan at foss dot arm dot com>
- Date: Thu, 29 Jun 2017 08:59:58 -0600
- Subject: Re: [PING][PATCH] Move the check for any_condjump_p from sched-deps to target macros
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx09.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx09.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=law at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 3014F6575F
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 3014F6575F
- References: <CO2PR07MB2694EE4FE1416466745550BC83330@CO2PR07MB2694.namprd07.prod.outlook.com> <CO2PR07MB2694D5ED2A06E8D85BD18789831E0@CO2PR07MB2694.namprd07.prod.outlook.com> <VI1PR0802MB2621A0766927C50FBBE1FAE6831E0@VI1PR0802MB2621.eurprd08.prod.outlook.com> <CO2PR07MB26946376EB24A0EB2BB66C6583110@CO2PR07MB2694.namprd07.prod.outlook.com> <675f27f3-2ea4-2c0b-84f2-c92c1289fbaf@redhat.com> <CO2PR07MB26944854CD014827C4E806DB83ED0@CO2PR07MB2694.namprd07.prod.outlook.com> <35251739-b27e-ea2b-fd1d-3bdd8ce799e7@redhat.com> <CO2PR07MB269308CC6FF5CD477A79D44383DC0@CO2PR07MB2693.namprd07.prod.outlook.com>
On 06/26/2017 10:19 PM, Hurugalawadi, Naveen wrote:
> Hi Jeff,
>
> Thanks for the review and your approval for final patch.
> Sorry, It was a long weekend and hence could not revert to your
> comments earlier.
>
>>> You need a ChangeLog entry, but I think that's it. Can you
>>> please repost with a ChangeLog entry for final approval?
>
> Please find the final patch and ChangeLog entry updated as required.
> Please review the same and let me know if its okay to commit?
>
> Thanks,
> Naveen
>
> 2017-06-27 Naveen H.S <Naveen.Hurugalawadi@cavium.com>
>
> * config/aarch64/aarch64.c (aarch_macro_fusion_pair_p): Push the
> check for CC usage into AARCH64_FUSE_CMP_BRANCH.
> * config/i386/i386.c (ix86_macro_fusion_pair_p): Push the check for
> CC usage from generic code to here.
> * sched-deps.c (sched_macro_fuse_insns): Move the condition for
> CC usage into the target macros.
>
OK for the trunk. Thanks for your patience.
Jeff