Re: varpool alias reorg

2011-06-28 Thread Richard Guenther
On Mon, 27 Jun 2011, Jan Hubicka wrote: > > On Fri, 24 Jun 2011, Jan Hubicka wrote: > > > > > Hi, > > > this is yet another variant of the fix. This time we stream builtins > > > decls as > > > usually, but at fixup time we copy the assembler names (if set) into the > > > builtin decls used by

Re: varpool alias reorg

2011-06-27 Thread Michael Matz
Hi, On Mon, 27 Jun 2011, Jan Hubicka wrote: > > I still like to stream unmodified builtins as builtins, as that is > > similar to pre-loading the streamer caches with things like > > void_type_node or sizetype. > > Doing so will need us to solve the other one decl rules probly. I didn't > rea

Re: varpool alias reorg

2011-06-27 Thread Jan Hubicka
> On Fri, 24 Jun 2011, Jan Hubicka wrote: > > > Hi, > > this is yet another variant of the fix. This time we stream builtins decls > > as > > usually, but at fixup time we copy the assembler names (if set) into the > > builtin decls used by folders. Not sure if it is any better than breaking >

Re: varpool alias reorg

2011-06-27 Thread Richard Guenther
On Fri, 24 Jun 2011, Jan Hubicka wrote: > Hi, > this is yet another variant of the fix. This time we stream builtins decls as > usually, but at fixup time we copy the assembler names (if set) into the > builtin decls used by folders. Not sure if it is any better than breaking > memops-asm, but I

Re: varpool alias reorg

2011-06-27 Thread Jan Hubicka
> > There are two problems here > > 1) We do not stream builtin decls and merge them outside lto-symtab (by > > just > > streaming references to builtins with their asm names). There is at > > least > > one extra PR related to this and on my TODO is to simply remove the > > code. >

Re: varpool alias reorg

2011-06-27 Thread Richard Guenther
On Thu, 23 Jun 2011, Jan Hubicka wrote: > > On Sat, Jun 18, 2011 at 7:19 AM, H.J. Lu wrote: > > > On Sat, Jun 18, 2011 at 1:32 AM, Jan Hubicka wrote: > > >> Hi, > > >> this patch makes symetric changes to varpool as did the prevoius series > > >> to cgraph. > > >> Basically the aliases are now

Re: varpool alias reorg

2011-06-24 Thread Jan Hubicka
Hi, this is yet another variant of the fix. This time we stream builtins decls as usually, but at fixup time we copy the assembler names (if set) into the builtin decls used by folders. Not sure if it is any better than breaking memops-asm, but I can imagine that things like glibc actually rename

Re: varpool alias reorg

2011-06-24 Thread Jan Hubicka
Hi, I also tested the attached variant that simply disable the builtins streaming. It fixes the testcase, too, bootstraps/regtestes x86_64 with and without plugin and builds mozilla. It also solves the decl sharing problems that leads to debug info confussion. However it breaks memops-asm.c testcas

Re: varpool alias reorg

2011-06-23 Thread Jan Hubicka
> On Sat, Jun 18, 2011 at 7:19 AM, H.J. Lu wrote: > > On Sat, Jun 18, 2011 at 1:32 AM, Jan Hubicka wrote: > >> Hi, > >> this patch makes symetric changes to varpool as did the prevoius series to > >> cgraph. > >> Basically the aliases are now represented as separate varpool nodes with > >> alia

Re: varpool alias reorg

2011-06-23 Thread H.J. Lu
On Sat, Jun 18, 2011 at 7:19 AM, H.J. Lu wrote: > On Sat, Jun 18, 2011 at 1:32 AM, Jan Hubicka wrote: >> Hi, >> this patch makes symetric changes to varpool as did the prevoius series to >> cgraph. >> Basically the aliases are now represented as separate varpool nodes with >> alias reference >>

Re: varpool alias reorg

2011-06-18 Thread H.J. Lu
On Sat, Jun 18, 2011 at 1:32 AM, Jan Hubicka wrote: > Hi, > this patch makes symetric changes to varpool as did the prevoius series to > cgraph. > Basically the aliases are now represented as separate varpool nodes with > alias reference > to the variable they refer to, with some infrastructure