This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch] OPT: Update heuristics for loop-invariant for address arithmetic.
- From: Richard Sandiford <rdsandiford at googlemail dot com>
- To: Ajit Kumar Agarwal <ajit dot kumar dot agarwal at xilinx dot com>
- Cc: "vmakarov\ at redhat dot com" <vmakarov at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Vinod Kathail <vinodk at xilinx dot com>, Shail Aditya Gupta <shailadi at xilinx dot com>, Vidhumouli Hunsigida <vidhum at xilinx dot com>, "Nagaraju Mekala" <nmekala at xilinx dot com>
- Date: Thu, 23 Apr 2015 20:09:55 +0100
- Subject: Re: [Patch] OPT: Update heuristics for loop-invariant for address arithmetic.
- Authentication-results: sourceware.org; auth=none
- References: <d3236939cb3e4429881fed98dbc192fe at BN1AFFO11FD054 dot protection dot gbl>
Very delayed answer, sorry...
Ajit Kumar Agarwal <ajit.kumar.agarwal@xilinx.com> writes:
> Hello All:
>
> The changes are made in the patch to update the heuristics for loop
> invariant for address arithemetic at RTL Level. The heuristics are
> updated with the consideration of single def and use for register
> pressure calculation instead Of ignoring it and also to update the
> estimated register pressure cost along with the check of actual uses
> with Address uses.
>
> With the above change, gains are seen in the Geomean for Mibench/EEMBC
> benchmarks for microblaze target. No Regression is seen in deja GNU
> regressions tests for microblaze.
Since thispatch is basically removing code, were you able to analyse why
that code was having a detrimental effect? I assume it benefited some
target originally.
Thanks,
Richard