[RFA] expand from SSA form (1/2)

Michael Matz matz@suse.de
Sun Apr 26 20:53:00 GMT 2009


Hi,

On Sun, 26 Apr 2009, Richard Guenther wrote:

> > into the var annotation (to be able to read it out again when the same
> > basevar is seen for a different partition).  But this whole info is
> > strictly local to the above function, so it doesn't need to live in the
> > annotation.  I could very well implement this as an array indexed by
> > DECL_UID.  The UIDs shouldn't become exceptionally large, so that seems
> 
> Hm.  DECL_UIDs are sparse (and global), it would be a bad idea to index
> an array with it.

Those few 100k entries ... nobody will notice.  Hmm, well, maybe someone 
does :-)

> Is var_ann->common.aux already used by out-of-SSA?

Oh joy.  This field is unused by out-of-SSA.  Which makes sense, as - 
drumroll - nothing (!) at all uses it (in the sense of deleting it doesn't 
break compile).

I'd rather remove that field and the fields in the var annotation.

> > feasible.  I wouldn't want to use a hash-table for fear of slowing down
> > var_map_base_init().
> 
> I think it would be not too bad ;)

I'll experiment a bit.


Ciao,
Michael.


More information about the Gcc-patches mailing list