This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ipa-cp] Do not insert call graph nodes into the queue multiple times
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>, Jan Hubicka <hubicka at ucw dot cz>
- Date: Thu, 23 Jul 2009 18:21:30 +0200
- Subject: Re: [PATCH, ipa-cp] Do not insert call graph nodes into the queue multiple times
- References: <20090723161023.GA27705@virgil.suse.cz>
> Hi,
>
> I have noticed that the work list we have in ipa-cp happily inserts
> the same node several times into itself even when there's no point in
> it. The patch below adds a flag to struct ipa_node_params so that we
> don't do it any more.
>
> Bootstrapped and tested on x86_64-linx.
>
> OK for trunk?
>
> Thanks,
>
> Martin
>
>
> 2009-07-23 Martin Jambor <mjambor@suse.cz>
>
> * ipa-prop.h (struct ipa_node_params): New flag node_enqued.
> (ipa_push_func_to_list_1): Declare.
> (ipa_push_func_to_list): New function.
>
> * ipa-prop.c (ipa_push_func_to_list_1): New function.
> (ipa_init_func_list): Call ipa_push_func_to_list_1.
> (ipa_push_func_to_list): Removed.
> (ipa_pop_func_from_list): Clear node_enqueued flag.
OK
Honza