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