Move default_def out of variable annotations

Jan Hubicka jh@suse.cz
Tue Dec 13 16:09:00 GMT 2005


> On Saturday 03 December 2005 13:51, Jan Hubicka wrote:
> 
> > 	* tree-flow-inline.h (set_default_def, default_def): Kill.
> > 	* tree-dfa.c (default_defs): New global variable.
> > 	(default_def, set_default_def): New functions.
> > 	* tree-ssa.c (init_tree_ssa, delete_tree_ssa): Add default_def hash.
> > 	* tree-flow.h (struct var_ann_d): Kill default_def field.
> > 	(set_default_def, default_def): Update prototype.
> > 	(default_defs): Declare.
> >
> Why didn't you stash DEFAULT_DEFS in CFUN?  The rest is fine with a few 

Because on IPA branch I have "ssa" structure for the SSA related stuff,
so I want to introduce that one first.  This patch is mostly meant to
reduce size of the main stuff that contains few controversal changes so
I expect some discussion about it first.

> > + /* Lookup UID in the default_defs hashtable and return the associated
> > +    variable.  */
> > +
> Didn't you mean VAR's UID?  default_defs should be capitalized.

Actually the comment is copied from similar logic dealing with
referenced vars.  I will update both and send updated patch.

Thanks,
Honza



More information about the Gcc-patches mailing list