This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM/AArch64][PR 60743] Reduce divider reservation duration in A53 pipeline decription
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, Bernd Edlinger <bernd dot edlinger at hotmail dot de>
- Date: Fri, 4 Apr 2014 16:41:46 +0100
- Subject: Re: [PATCH][ARM/AArch64][PR 60743] Reduce divider reservation duration in A53 pipeline decription
- Authentication-results: sourceware.org; auth=none
- References: <533EA72A dot 7010109 at arm dot com>
- Reply-to: ramrad01 at arm dot com
On Fri, Apr 4, 2014 at 1:35 PM, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
> Hi all,
>
> In PR 60743 it is noted that the genautomata computation has increased a lot
> in both size and time due to my recently added a53 scheduling additions.
> This patch attempts to mitigate that by reducing the large reservation
> duration of the dividers that is causing a state-space explosion in the
> automaton.
> Without this patch I've observed a memory usage of around 750MB on my host
> machine while running genautomata. With this patch, it is about 410MB and
> much faster.
>
> Bernd, can you try this out and see if the genautomata behaviour is
> acceptable on your system now?
>
> Bootstrapped and tested on aarch64-linux-gnu and arm-none-linux-gnueabihf.
> Looked at various testcases involving the dividers to make sure that codegen
> is not affected.
>
> Ok for trunk?
This is ok. Thanks for fixing this so promptly.
Ramana
> Thanks,
> Kyrill
>
> 2014-04-04 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
>
> PR bootstrap/60743
> * config/arm/cortex-a53.md (cortex_a53_fdivs): Reduce reservation
> duration.
> (cortex_a53_fdivd): Likewise.