Correct fix for scheduler bug PR11320

Bernd Schmidt bernds@codesourcery.com
Thu Jul 14 11:42:00 GMT 2011


On 07/14/11 13:20, Andrey Belevantsev wrote:
> Hello Bernd,
> 
> FWIW, we have discussed your change with Alexander and we think you are
> right about the scheduler changes.  One question is:
> 
> On 14.07.2011 14:03, Bernd Schmidt wrote:
>> --- gcc/sched-deps.c    (revision 176195)
>> +++ gcc/sched-deps.c    (working copy)
>> @@ -568,7 +568,7 @@
>>        (rev1==rev2
>>        ? reversed_comparison_code (cond2, NULL)
>>        : GET_CODE (cond2))
>> -      && XEXP (cond1, 0) == XEXP (cond2, 0)
>> +      && rtx_equal_p (XEXP (cond1, 0), XEXP (cond2, 0))
>>        && XEXP (cond1, 1) == XEXP (cond2, 1))
>>      return 1;
>>    return 0;
> 
> this hunk from conditions_mutex_p seems to be unrelated?

Oh yes, sorry about that. That was approved a while ago and I haven't
gotten around to checking it in.


Bernd



More information about the Gcc-patches mailing list