This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: v2 patch to fix calculix degradation
- From: Luis Machado <luisgpm at linux dot vnet dot ibm dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, Jeff Law <law at redhat dot com>, "H.J. Lu" <hjl dot tools at gmail dot com>, Kenneth Zadeck <Kenneth dot Zadeck at NaturalBridge dot com>
- Date: Tue, 16 Dec 2008 13:29:17 -0200
- Subject: Re: RFA: v2 patch to fix calculix degradation
- References: <492631A2.1070506@redhat.com>
- Reply-to: luisgpm at linux dot vnet dot ibm dot com
Hi Vladimir,
I've noticed a 3% degradation on SPEC2K's facerec introduced by this
patch on PPC64. I'm still going through it to see what has changed.
Regards,
Luis
On Thu, 2008-11-20 at 22:57 -0500, Vladimir Makarov wrote:
> I fixed the problem reported by H.J. in
>
> http://gcc.gnu.org/ml/gcc-patches/2008-11/msg00942.html
>
> The patch was successfully bootstrapped (with all enabled languages) and
> tested on x86, x86_64, ppc64, an itanium.
>
> Ok to commit?
>
> 2008-11-19 Vladimir Makarov <vmakarov@redhat.com>
>
> * doc/invoke.texi (ira-max-loops-num): Change semantics.
>
> * ira-int.h (struct ira_loop_tree_node): New member to_remove_p.
>
> * ira-color.c (allocno_spill_priority): New function.
> (remove_allocno_from_bucket_and_push, push_allocno_to_spill):
> Print more info about the spilled allocno.
> (push_allocnos_to_stack): Use allocno_spill_priority. Add more
> checks on bad spill.
>
> * ira-build.c (loop_node_to_be_removed_p): Remove.
> (loop_compare_func, mark_loops_for_removal): New functions.
> (remove_uneccesary_loop_nodes_from_loop_t): Use member
> to_remove_p.
> (remove_unnecessary_allocnos): Call mark_loops_for_removal.
>
> * ira.c (ira): Don't change flag_ira_algorithm.
>
> * params.def (ira-max-loops-num): Change the value.
>
>