Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-06-03 Thread Maxim Kuvyrkov
On Jun 2, 2014, at 7:54 PM, Andreas Schwab wrote: > Regtested on ia64-suse-linux and installed as obvious. > > Andreas. > > * config/ia64/ia64.c > (ia64_first_cycle_multipass_dfa_lookahead_guard): Check > pending_data_specs first. > > diff --git a/gcc/config/ia64/ia64.c b/gcc

Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-06-02 Thread Andreas Schwab
Regtested on ia64-suse-linux and installed as obvious. Andreas. * config/ia64/ia64.c (ia64_first_cycle_multipass_dfa_lookahead_guard): Check pending_data_specs first. diff --git a/gcc/config/ia64/ia64.c b/gcc/config/ia64/ia64.c index 118e5bf..4c5390b 100644 --- a/gcc/conf

Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-06-01 Thread Andreas Schwab
Maxim Kuvyrkov writes: > @@ -7535,28 +7530,24 @@ ia64_variable_issue (FILE *dump ATTRIBUTE_UNUSED, > can be chosen. */ > > static int > -ia64_first_cycle_multipass_dfa_lookahead_guard (rtx insn) > +ia64_first_cycle_multipass_dfa_lookahead_guard (rtx insn, int ready_index) > { >gcc_as

Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-05-23 Thread Jeff Law
On 05/23/14 01:35, Maxim Kuvyrkov wrote: On May 23, 2014, at 7:23 PM, Andreas Schwab wrote: ../../gcc/config/ia64/ia64.c: In function 'int ia64_first_cycle_multipass_dfa_lookahead_guard(rtx, int)': ../../gcc/config/ia64/ia64.c:7551:1: error: control reaches end of non-void function [-Werror=

Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-05-23 Thread Maxim Kuvyrkov
On May 23, 2014, at 7:23 PM, Andreas Schwab wrote: > ../../gcc/config/ia64/ia64.c: In function 'int > ia64_first_cycle_multipass_dfa_lookahead_guard(rtx, int)': > ../../gcc/config/ia64/ia64.c:7551:1: error: control reaches end of non-void > function [-Werror=return-type] Fixed, sorry about the

Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-05-23 Thread Andreas Schwab
../../gcc/config/ia64/ia64.c: In function 'int ia64_first_cycle_multipass_dfa_lookahead_guard(rtx, int)': ../../gcc/config/ia64/ia64.c:7551:1: error: control reaches end of non-void function [-Werror=return-type] Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7

Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-05-20 Thread David Edelsohn
On Tue, May 20, 2014 at 5:46 PM, Pat Haugen wrote: > On 05/19/2014 11:27 PM, Maxim Kuvyrkov wrote: >> >> Changes to ia64 and rs6000 are mostly mechanical to update hook return >> values, but port maintainers may wish to review those. > > I'm not a maintainer, but the rs6000 changes look good to me

Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-05-20 Thread Pat Haugen
On 05/19/2014 11:27 PM, Maxim Kuvyrkov wrote: Changes to ia64 and rs6000 are mostly mechanical to update hook return values, but port maintainers may wish to review those. I'm not a maintainer, but the rs6000 changes look good to me. -Pat

Re: [PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-05-20 Thread Jeff Law
On 05/19/14 22:27, Maxim Kuvyrkov wrote: Hi, This patch cleans up haifa-sched.c:choose_ready() function while allow more powerful customization by backends at the same time. The primary change is that targetm.sched.first_cycle_multipass_dfa_lookahead_guard hook is converted from returning a boo

[PATCH, sched] Cleanup and improve multipass_dfa_lookahead_guard

2014-05-19 Thread Maxim Kuvyrkov
Hi, This patch cleans up haifa-sched.c:choose_ready() function while allow more powerful customization by backends at the same time. The primary change is that targetm.sched.first_cycle_multipass_dfa_lookahead_guard hook is converted from returning a boolean OK/Nada value to returning an actio