This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: regrename speedup
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Bernd Schmidt <bernds_cb1 at t-online dot de>
- Cc: Eric Botcazou <ebotcazou at adacore dot com>, gcc-patches at gcc dot gnu dot org
- Date: Fri, 27 Nov 2009 16:49:43 -0800
- Subject: Re: regrename speedup
- References: <4AD9CEF2.50908@t-online.de> <200910221504.37462.ebotcazou@adacore.com> <4AFC5322.1010308@t-online.de>
On Thu, Nov 12, 2009 at 10:25 AM, Bernd Schmidt <bernds_cb1@t-online.de> wrote:
> Eric Botcazou wrote:
>>> ? ? ?PR rtl/38582
>>
>> PR rtl-opt (or PR rtl-optimization).
>>
>>> ? ? ?* regrename.c (struct du_head): New structure; some elements moved
>>> ? ? ?from...
>>> ? ? ?(struct du_chain): ... this one.
>>> ? ? ?(open_chains, closed_chains): Now of type struct du_head *.
>>> ? ? ?(do_replace): Accept du_head argument, not du_chain. ?All callers
>>> ? ? ?changed. ?Modified code to match new data structures.
>>> ? ? ?(build_def_use): Return a list of du_head structures. ?Modified code
>>> ? ? ?to match new data structures.
>>> ? ? ?(dump_def_use_chain): Accept du_head argument, not du_chain. ?All
>>> ? ? ?callers changed. ?Modified code to match new data structures.
>>> ? ? ?(merge_overlapping_regs): Accept du_head argument, not du_chain. ?All
>>> ? ? ?callers changed. ?Modified code to match new data structures.
>>> ? ? ?(scan_rtx_reg): Change type of this_regno and this_nregs to unsigned.
>>> ? ? ?Allocate a du_head structure as well as a du_chain when creating a
>>> ? ? ?new chain. ?Modified other code to match new data structures.
>>
>> OK with the trailing spaces removed in
> [...]
>
> Thanks, committed now (vacation got in the way). ?I'll post a followup
> patch as well which takes care of the remaining performance problem.
>
>
This caused:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42202
--
H.J.