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
On Tue, Nov 25, 2008 at 8:01 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Tue, Nov 25, 2008 at 10:42 AM, Jeff Law <law@redhat.com> wrote:
>> 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.
>>>
>>>
>> OK.
>>
>
> Hi Vladimir,
>
> This patch caused:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38272
>
> It may have something to do with
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38254
It also breaks SPEC CPU 2006:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38280
--
H.J.