This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH PR65161]


Here is updated patch accordingly to Alexander comments.

BTW another function using HID interface is do_reorder_for_imul and it
is called from ix86_sched_reorder.

Is it OK for trunk?

2015-02-25 13:26 GMT+03:00 Alexander Monakov <amonakov@ispras.ru>:
>
>
> On Wed, 25 Feb 2015, Yuri Rumyantsev wrote:
>
>> Hi All,
>>
>> I prepared new patch which includes test-case.
>>
>> I can't agree with patch proposed by Alexander since other functions
>> doing ready list reordering also use HID interface, so I put escape
>> check in ix86_sched_reorder.
>
> I don't see how that is the case.  Can you point me to specific lines of code
> in ix86_sched_reorder or its callees that access HID?
>
> In any case please use sel_sched_p () rather than flag_selective_scheduling2.
>
> Thanks.
>
> Alexander

Attachment: patch.2
Description: Binary data


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]