Re: Stream ODR types

2014-09-22 Thread Richard Biener
On Wed, 17 Sep 2014, Jan Hubicka wrote: > Hi, > this patch renames types reported by Wodr during LTO bootstrap. > > Bootrapping/regtesting in progress, OK if it passes? > > Honza > > * tree-ssa-ccp.c (prop_value_d): Rename to ... > (ccp_prop_value_t): ... this one to avoid ODR viola

Re: Stream ODR types

2014-09-17 Thread Jan Hubicka
Hi, this patch renames types reported by Wodr during LTO bootstrap. Bootrapping/regtesting in progress, OK if it passes? Honza * tree-ssa-ccp.c (prop_value_d): Rename to ... (ccp_prop_value_t): ... this one to avoid ODR violation; update uses. * ipa-prop.c (struct type_ch

Re: Stream ODR types

2014-09-15 Thread Richard Biener
On Fri, 12 Sep 2014, Jan Hubicka wrote: > > > For ODR warnings and TBAA I think i want other types, too. But yep, we > > > need to handle > > > gracefuly component types that does not have names and we could drop > > > names of types > > > and handle them as component types as it seems fit. > >

Re: Stream ODR types

2014-09-12 Thread Jan Hubicka
> > For ODR warnings and TBAA I think i want other types, too. But yep, we > > need to handle > > gracefuly component types that does not have names and we could drop names > > of types > > and handle them as component types as it seems fit. > > > > OK, so if you agree, I will go ahead with thi

Re: Stream ODR types

2014-09-12 Thread Richard Biener
On Thu, 11 Sep 2014, Jan Hubicka wrote: > > On Thu, 11 Sep 2014, Jan Hubicka wrote: > > > > > Hi, > > > this patch adds computation and streaming of mangled type names. As > > > suggested by Jason, > > > it simple calls DECL_ASSEMBLER_NAME on all names types and lets C++ > > > supply them. > >

Re: Stream ODR types

2014-09-12 Thread Richard Biener
On Thu, 11 Sep 2014, Jason Merrill wrote: > On 09/11/2014 03:06 AM, Jan Hubicka wrote: > > http://kam.mff.cuni.cz/~hubicka/odr-warnings-firefox.txt > > > /aux/hubicka/firefox4/content/media/fmp4/ffmpeg/libav53/include/libavcodec/avcodec.h:997:0: > > note: the first difference of corresponding def

Re: Stream ODR types

2014-09-11 Thread Jan Hubicka
> On 09/11/2014 03:06 AM, Jan Hubicka wrote: > >http://kam.mff.cuni.cz/~hubicka/odr-warnings-firefox.txt > > >/aux/hubicka/firefox4/content/media/fmp4/ffmpeg/libav53/include/libavcodec/avcodec.h:997:0: > > note: the first difference of corresponding definitions is field ‘data’ > > uint8_t *dat

Re: Stream ODR types

2014-09-11 Thread Jan Hubicka
> On Thu, 11 Sep 2014, Jan Hubicka wrote: > > > Hi, > > this patch adds computation and streaming of mangled type names. As > > suggested by Jason, > > it simple calls DECL_ASSEMBLER_NAME on all names types and lets C++ supply > > them. > > This makes it possible to stablish precise ODR type eq

Re: Stream ODR types

2014-09-11 Thread Jason Merrill
On 09/11/2014 03:06 AM, Jan Hubicka wrote: http://kam.mff.cuni.cz/~hubicka/odr-warnings-firefox.txt /aux/hubicka/firefox4/content/media/fmp4/ffmpeg/libav53/include/libavcodec/avcodec.h:997:0: note: the first difference of corresponding definitions is field ‘data’ uint8_t *data[AV_NUM_DAT

Re: Stream ODR types

2014-09-11 Thread Richard Biener
On Thu, 11 Sep 2014, Jan Hubicka wrote: > Hi, > this patch adds computation and streaming of mangled type names. As > suggested by Jason, > it simple calls DECL_ASSEMBLER_NAME on all names types and lets C++ supply > them. > This makes it possible to stablish precise ODR type equivalency at LTO