Re: RFA: v2 patch to fix calculix degradation

["H.J. Lu" <hjl dot tools at gmail dot com>]

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


-- 
H.J.