This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly

From: Uros Bizjak <ubizjak at gmail dot com>
To: Richard Biener <richard dot guenther at gmail dot com>
Cc: Eric Botcazou <ebotcazou at adacore dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Evgeny Stupachenko <evstupac at gmail dot com>, Jan Hubicka <hubicka at ucw dot cz>
Date: Sat, 22 Nov 2014 20:03:23 +0100
Subject: Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly
Authentication-results: sourceware.org; auth=none
References: <CAOvf_xwBqY++PGRC_+1=rzHO18jvc+TP5mmR=PjYvGgm=63NuA at mail dot gmail dot com> <32201320 dot a2T5pL3L4j at polaris> <CAFULd4ZoWL-qR_LQgGJ0ExTKdaPJxA0Gi3dVzbVZLsRyfPnr8A at mail dot gmail dot com> <1871210 dot X8Ouu1Z02f at polaris> <507E7F44-3CB4-49FA-A715-DAF513DCDF0D at gmail dot com>

On Sat, Nov 22, 2014 at 7:38 PM, Richard Biener
<richard.guenther@gmail.com> wrote:
> On November 22, 2014 12:24:22 PM CET, Eric Botcazou <ebotcazou@adacore.com> wrote:
>>> Yeah, but after a couple of pings for a generic change, we went the
>>target
>>> way.
>>
>>That's a bit of a shame, the 400 -> 100 change was very likely tested
>>only on
>>x86-64 and nevetheless applied to the generic code, so the fix
>>repairing the
>>damages should also be applied to the generic code.
>
> A patch to bump the generic limit is OK.
>
> Targets that dont want it can reduce it in target specific code.

I have committed the attached patch:

2014-11-22  Uros Bizjak  <ubizjak@gmail.com>

    * params.def (PARAM_MAX_COMPLETELY_PEELED_INSNS): Increase to 200.
    * config/i386/i386.c (ix86_option_override_internal): Do not increase
    PARAM_MAX_COMPLETELY_PEELED_INSNS.

Bootstrapped on x86_64-linux-gnu.

Uros.

Attachment: p.diff.txt
Description: Text document

References:
- Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly
  - From: Eric Botcazou
- Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly
  - From: Uros Bizjak
- Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly
  - From: Eric Botcazou
- Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly
  - From: Richard Biener

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]