This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: patch for PR37488
- From: Kenneth Zadeck <zadeck at naturalbridge dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, Jeffrey Law <law at redhat dot com>, Kenneth Zadeck <Kenneth dot Zadeck at naturalbridge dot com>
- Date: Wed, 22 Jul 2009 09:43:48 -0400
- Subject: Re: RFA: patch for PR37488
- References: <4A4D2C3C.8010402@redhat.com>
Vladimir Makarov wrote:
> The following patch solves PR37488. The details of the problem can
> be found on
>
This patch is fine.
kenny
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37488
>
> The patch was successfully bootstrapped and tested on x86 and x86_64.
> It was benchmarked on SPEC2000 on x86 with -O2 -march=i686
> -mfpmath=sse -msse2. The average scores are practically the same.
>
> Ok to commit?
>
> 2009-07-02 Vladimir Makarov <vmakarov@redhat.com>
>
> PR target/37488
> * ira-lives.c (bb_has_abnormal_call_pred): New function.
> (process_bb_node_lives): Use it.
>
> * ira.c (setup_cover_and_important_classes): Don't setup
> ira_important_class_nums. Add cover classes to the end of
> important classes.
> (cover_class_order, comp_reg_classes_func,
> reorder_important_classes): New.
> (find_reg_class_closure): Use reorder_important_classes.
>
> * config/i386/i386.h (IRA_COVER_CLASSES): Remove.
>
> * config/i386/i386.c (i386_ira_cover_classes): New function.
> (TARGET_IRA_COVER_CLASSES): Redefine.
>
> * doc/tm.texi (TARGET_IRA_COVER_CLASSES): Add a comment about
> importance of order of cover classes in the array.
>