On 09/03/14 02:45, Martin Jambor wrote:
Hi,

this is a followup to my previous PR-fixing patch.  At ever more
places we currently do

   if (!node->expand_thunk (false, whatever))
     node->analyzed = true;

and we always set the flag when expand_thunk returns with false (it
only can when the first parameter is false).  So I thought it would be
much nicer to set the analyzed flag in expand_thunk itself when it
returns false, especially given that we probably want to set the flag
at as few places as reasonably possible.

Bootstrapped and tested on x86_64-linux.  OK for trunk?

Thanks,

Martin


2014-09-01  Martin Jambor  <mjam...@suse.cz>

        * cgraphunit.c (expand_thunk): If not expanding, set analyzed flag.
        (analyze): Do not set analyze flag if expand_thunk returns false;.
        (create_wrapper): Likewise.
        * cgraphclones.c (duplicate_thunk_for_node): Likewise.
OK.
Jeff

Reply via email to