On Sun, Dec 13, 2009 at 15:51, Richard Guenther <rguent...@suse.de> wrote:

> + /* ???  We could free non-constant DECL_SIZE, DECL_SIZE_UNIT
> +    and DECL_FIELD_OFFSET.  But it's cheap enough to not do
> +    that and refrain from adding workarounds to dwarf2out.c  */
> +
> + /* DECL_FCONTEXT is only used for debug info generation.  */
> + if (TREE_CODE (decl) == FIELD_DECL
> +     && debug_info_level < DINFO_LEVEL_TERSE)
> +   DECL_FCONTEXT (decl) = NULL_TREE;
> +

Yes, keeping a reminder here will help future work in early debug
generation.  As for the rest of the patch, anything that gets us
closer to a working -g -flto is better than nothing.  Not clearing
these fields should not be a big deal for now.

The patch is OK with the reminders that Michael suggested.


Diego.

Reply via email to