[lto][patch] Remove after_inlining flag

Diego Novillo dnovillo@google.com
Thu Oct 16 13:39:00 GMT 2008


On Thu, Oct 16, 2008 at 08:14, Rafael Espindola <espindola@google.com> wrote:
> This flag is causing me troubles trying to run the inliner is
> cc1/cc1plus with -flto. I am testing the patch on trunk to get better
> coverage. OK for the branch if the tests are OK?
>
> 2008-10-16 Rafael Espindola  <espindola@google.com>
>
>        * except.c (reachable_next_level): Don't use cfun->after_inlining.
>        * function.h (struct function): Remove after_inlining.
>        * tree-inline.c (initialize_cfun): Don't copy cfun->after_inlining.
>        (expand_call_inline): Ignore cfun->after_inlining.
>        * tree-optimize.c (execute_fixup_cfg): Don't set cfun->after_inlining.
>        * tree-ssa-live.c (remove_unused_scope_block_p): Ignore
>        cfun->after_inlining.

What kind of troubles?  I'm not really sure what this flag is used
for, so I can't say whether it's safe to remove.  It doesn't seem that
we use it for much, and the places that do use it are not very clear
about its purpose.

In principle I wouldn't mind removing it, but I would like to
understand why exactly.

Jan, any insights?


Thanks.  Diego.



More information about the Gcc-patches mailing list