Put scope blocks on a diet

Alexandre Oliva aoliva@redhat.com
Wed Oct 3 17:01:00 GMT 2007


On Aug  6, 2007, Jan Hubicka <jh@suse.cz> wrote:

> I would like to test the attached patch.  It prevents the removal before
> inlining so UIDs should remain the same.  However I do get massive
> regressions on bootstrap-debug with or without the block removal code.
> I will now rebuild and see if the set of failing files changes by 
> disabling block removal, but if you do have some testcases handy, it
> would help.

> I wonder if we need to keep those at all - dwarf2 already uses abstract
> origin to avoid need to saveinfo on variables in each copy.  We can
> probably just keep around the dead variables on separate list in their
> ABSTRACT_ORIGIN -g or not?

This patch of yours (except for the ChangeLog entry) fixes the
-g-changes-generated-code in the trunk, so this should go in.

However, it breaks the vta branch, because we still keep debug
annotations for variables in blocks that your patch removes.  I
suppose I could just drop such annotations in the floor for now, but,
in the long run, should I?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: gcc-debug-unused-scope-block-alt.patch
Type: text/x-patch
Size: 884 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20071003/f2ea2ce6/attachment.bin>
-------------- next part --------------

-- 
Alexandre Oliva         http://www.lsd.ic.unicamp.br/~oliva/
FSF Latin America Board Member         http://www.fsfla.org/
Red Hat Compiler Engineer   aoliva@{redhat.com, gcc.gnu.org}
Free Software Evangelist  oliva@{lsd.ic.unicamp.br, gnu.org}


More information about the Gcc-patches mailing list