[PATCH] Fix PR 52203
Alexander Monakov
amonakov@ispras.ru
Wed Mar 7 11:46:00 GMT 2012
On Wed, 7 Mar 2012, Andrey Belevantsev wrote:
> Hello,
>
> This PR is again about insns that are recog'ed as >=0 but do not change the
> processor state. As explained in
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52203#c8, I've tried experimenting
> with an attribute marking those "empty" insns in MD files and asserting that
> all other insns do have reservations. As this doesn't seem to be interesting,
> I give up with the idea, and the below patch makes sel-sched do exactly what
> the Haifa scheduler does, i.e. do not count such insns against issue_rate when
> modelling clock cycles.
>
> Tested on ia64 and x86-64, OK for trunk? No testcase again because of the
> amount of flags needed.
>
> Andrey
>
> 2012-03-07 Andrey Belevantsev <abel@ispras.ru>
>
> PR rtl-optimization/52203
> * sel-sched.c (estimate_insn_cost): New parameter pempty. Adjust
> all callers to pass NULL except ...
> (reset_sched_cycles_in_current_ebb): ... here, save the value
> in new variable 'empty'. Increase issue_rate only for
> non-empty insns.
This is OK.
Thanks.
--
Alexander
More information about the Gcc-patches
mailing list