Re: Silence merge warnings on artiical types

2015-03-30 Thread Jason Merrill
On 03/30/2015 01:23 PM, Jan Hubicka wrote: Jason probably knows better, but I think only real C++ types comply the One Defintion Type and should be merged. Anything we create artifically in compiler is probably not covered by this. Agreed, compiler internals are outside the scope of the langu

Re: Silence merge warnings on artiical types

2015-03-30 Thread Jan Hubicka
> On Mon, Mar 30, 2015 at 07:06:39PM +0200, Jan Hubicka wrote: > > > On Mon, Mar 30, 2015 at 05:02:57 +0200, Jan Hubicka wrote: > > > > * lto-symtab.c (lto_symtab_merge_decls_2): Silence warnings on > > > > artificial decls. > > > > > > Shouldn't this patch fix libgomp.c++/target-3

Re: Silence merge warnings on artiical types

2015-03-30 Thread Jakub Jelinek
On Mon, Mar 30, 2015 at 07:06:39PM +0200, Jan Hubicka wrote: > > On Mon, Mar 30, 2015 at 05:02:57 +0200, Jan Hubicka wrote: > > > * lto-symtab.c (lto_symtab_merge_decls_2): Silence warnings on > > > artificial decls. > > > > Shouldn't this patch fix libgomp.c++/target-3.C in an offloading-enab

Re: Silence merge warnings on artiical types

2015-03-30 Thread Jan Hubicka
> On Mon, Mar 30, 2015 at 19:06:39 +0200, Jan Hubicka wrote: > > > On Mon, Mar 30, 2015 at 05:02:57 +0200, Jan Hubicka wrote: > > > > * lto-symtab.c (lto_symtab_merge_decls_2): Silence warnings on > > > > artificial decls. > > > > > > Shouldn't this patch fix libgomp.c++/target-3.C

Re: Silence merge warnings on artiical types

2015-03-30 Thread Ilya Verbin
On Mon, Mar 30, 2015 at 19:06:39 +0200, Jan Hubicka wrote: > > On Mon, Mar 30, 2015 at 05:02:57 +0200, Jan Hubicka wrote: > > > * lto-symtab.c (lto_symtab_merge_decls_2): Silence warnings on > > > artificial decls. > > > > Shouldn't this patch fix libgomp.c++/target-3.C in an offloading-enable

Re: Silence merge warnings on artiical types

2015-03-30 Thread Jan Hubicka
> On Mon, Mar 30, 2015 at 05:02:57 +0200, Jan Hubicka wrote: > > * lto-symtab.c (lto_symtab_merge_decls_2): Silence warnings on > > artificial decls. > > Shouldn't this patch fix libgomp.c++/target-3.C in an offloading-enabled > configuration? It still fails... > > libgomp/testsuite/libg

Re: Silence merge warnings on artiical types

2015-03-30 Thread Ilya Verbin
On Mon, Mar 30, 2015 at 05:02:57 +0200, Jan Hubicka wrote: > * lto-symtab.c (lto_symtab_merge_decls_2): Silence warnings on > artificial decls. Shouldn't this patch fix libgomp.c++/target-3.C in an offloading-enabled configuration? It still fails... libgomp/testsuite/libgomp.c++/../l

Re: Silence merge warnings on artiical types

2015-03-30 Thread Jan Hubicka
> On Mon, 30 Mar 2015, Jan Hubicka wrote: > > > Hi, > > when compiling C++ program that define different number of virtual method > > in different classes, we output warnings about their virtual tables > > being of different type. THese warnings looks ugly and we are able > > to diagnose the situa

Re: Silence merge warnings on artiical types

2015-03-30 Thread Richard Biener
On Mon, 30 Mar 2015, Jan Hubicka wrote: > Hi, > when compiling C++ program that define different number of virtual method > in different classes, we output warnings about their virtual tables > being of different type. THese warnings looks ugly and we are able > to diagnose the situation in more s

Silence merge warnings on artiical types

2015-03-29 Thread Jan Hubicka
Hi, when compiling C++ program that define different number of virtual method in different classes, we output warnings about their virtual tables being of different type. THese warnings looks ugly and we are able to diagnose the situation in more sensible way. This patch simply silences all warnin