Re: Clear more useless flags

2018-11-07 Thread Richard Biener
On Wed, 7 Nov 2018, Jan Hubicka wrote: > > > + /* TREE_PUBLIC is used to tell if type is anonymous. */ > > > + DECL_EXTERNAL (decl) = 0; > > > + TYPE_DECL_SUPPRESS_DEBUG (decl) = 0; > > > > DECL_EXTERNAL and TYPE_DECL_SUPPRESS_DEBUG map to the same decl_flag_1 ... > > so I'd say y

Re: Clear more useless flags

2018-11-07 Thread Jan Hubicka
> > + /* TREE_PUBLIC is used to tell if type is anonymous. */ > > + DECL_EXTERNAL (decl) = 0; > > + TYPE_DECL_SUPPRESS_DEBUG (decl) = 0; > > DECL_EXTERNAL and TYPE_DECL_SUPPRESS_DEBUG map to the same decl_flag_1 ... > so I'd say you should use TYPE_DECL_SUPPRESS_DEBUG only here. I

Re: Clear more useless flags

2018-11-07 Thread Richard Biener
On Wed, 7 Nov 2018, Jan Hubicka wrote: > Hi, > this patch enables bit more merging by clearing more flags that are > unnecesary and differ in practice across different copies of same ODR > types. > > lto-bootstrapped/regtested x86_64-linux, OK? > * tree.c (fld_incomplete_type_of): Clear TRE