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 violation;

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

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. OK, so if

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

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. This makes it

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 this patch and

Stream ODR types

2014-09-11 Thread Jan Hubicka
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 (till now we can do that only for complete class

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

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

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 equivalency at

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