This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][4.4] Add a global DECL_UID -> tree mapping
- From: "Richard Guenther" <richard dot guenther at gmail dot com>
- To: "Diego Novillo" <dnovillo at google dot com>
- Cc: "Andrew MacLeod" <amacleod at redhat dot com>, "Richard Guenther" <rguenther at suse dot de>, gcc-patches at gcc dot gnu dot org
- Date: Sat, 23 Feb 2008 13:48:59 +0100
- Subject: Re: [PATCH][4.4] Add a global DECL_UID -> tree mapping
- References: <Pine.LNX.4.64.0711091532070.4164@zhemvz.fhfr.qr> <Pine.LNX.4.64.0802221415050.5150@zhemvz.fhfr.qr> <47BF1BCF.8010300@redhat.com> <47BF5EEB.2030107@google.com>
On Sat, Feb 23, 2008 at 12:46 AM, Diego Novillo <dnovillo@google.com> wrote:
> On 2/22/08 2:00 PM, Andrew MacLeod wrote:
>
> > This oughta fix the odd compile time hit we have when the hash table
> > doesn't perform well too eh? I remember looking at something a few weeks
> > ago where most of the operand scan time (which was needlessly excessive)
> > was almost all in referenced_var hash table lookups. getting rid of
> > that hash lookup is a good thing.
Though we replace it with a hash lookup on the global table.
> Indeed. I like this change quite a bit. Thanks for working on this
> Richard.
It might also prove useful for IPA analysis where you now can track vars
with just their UIDs (rather than function + UID).
Richard.