This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR ipa/65087
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Martin Liška <mliska at suse dot cz>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, "hubicka@ >> Jan Hubicka" <hubicka at ucw dot cz>, Martin Jambor <mjambor at suse dot cz>
- Date: Mon, 2 Mar 2015 18:20:04 +0100
- Subject: Re: [PATCH] Fix PR ipa/65087
- Authentication-results: sourceware.org; auth=none
- References: <54F49553 dot 7030302 at suse dot cz>
> Hello.
>
> Following one line patch is fix for PR ipa/65087. No regression has been
> seen on x86_64-linux-pc.
OK, it won't happend large programs, but can you arrange its execute method to
return TODO_remove_functions only if some merging actually happened?
Patch is preapproved with this change.
Honza
>
> Ready for trunk?
> Thanks,
> Martin
> >From f39be56c66387c698546c498550d38512ddeb477 Mon Sep 17 00:00:00 2001
> From: mliska <mliska@suse.cz>
> Date: Mon, 2 Mar 2015 16:56:42 +0100
> Subject: [PATCH] Fix PR ipa/65087.
>
> gcc/ChangeLog:
>
> 2015-03-02 Martin Liska <mliska@suse.cz>
> Martin Jambor <mjambor@suse.cz>
>
> PR ipa/65087
> * ipa-icf.c: Add TODO_remove_functions to the IPA pass.
> ---
> gcc/ipa-icf.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/ipa-icf.c b/gcc/ipa-icf.c
> index 31fcbec..56f59a9 100644
> --- a/gcc/ipa-icf.c
> +++ b/gcc/ipa-icf.c
> @@ -3049,7 +3049,7 @@ const pass_data pass_data_ipa_icf =
> 0, /* properties_provided */
> 0, /* properties_destroyed */
> 0, /* todo_flags_start */
> - 0, /* todo_flags_finish */
> + TODO_remove_functions, /* todo_flags_finish */
> };
>
> class pass_ipa_icf : public ipa_opt_pass_d
> --
> 2.1.2
>