On Sat, Jan 16, 2016 at 11:31:49AM +0100, Richard Biener wrote:
> On January 16, 2016 11:27:09 AM GMT+01:00, Jan Hubicka <hubi...@ucw.cz> wrote:
> >> On Fri, 15 Jan 2016, Jan Hubicka wrote:
> >>
> >> > Hi,
> >> > this is last of my patches to improve partitionability of programs.
> >
> >> > This patch compacts BLOCK_VARs lists and trows away TYPE_DECLs for
> >-g0
> >> > and redundant TYPE_DECLs on all levels. It does make noticeable
> >difference
> >> > on firefox, but I managed to erase the numbers (can re-test them if
> >requested)
> >> >
> >> > Bootstrapped/regtested x86_64-linux, OK?
> >>
> >> Hmm. So I wonder why remove_unused_locals does not do this then
> >> or if it does not because it would change code-gen based on -g/-g0
> >> I wonder why your patch wouldn't introduce that issue with LTO.
> >
> >Usually the things stay to make DECL_UID stable across -g and -g0.
>
> Only DECL_UID order needs to be stable I think.
There is certainly something we only care about order and not exact numbers,
but I'm not sure if it is DECL_UIDs - otherwise we'd happily use
allocate_decl_uid ()
even for DECL_DEBUG_EXPRs, but we instead use a different counter for those.
Alex?
Jakub
