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