Sandra Loosemore <san...@codesourcery.com> writes: > This is another patch that has been present in our local source base for some > years now. It originally came from MIPS; I've verified that we have legal > permission to contribute it to the FSF. > > The 74k.md parts of this patch depend on the not-yet-reviewed "74k madd > scheduler > tweaks" patch I posted the other day: > > http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00062.html > > Assuming that one gets approved, is this patch OK for mainline?
OK with: > +/* DSP ALU can bypass data with no delays for the following pairs. */ > +enum insn_code dspalu_bypass_table[][2] = > +{ > + {CODE_FOR_mips_addsc, CODE_FOR_mips_addwc}, > + {CODE_FOR_mips_cmpu_eq_qb, CODE_FOR_mips_pick_qb}, > + {CODE_FOR_mips_cmpu_lt_qb, CODE_FOR_mips_pick_qb}, > + {CODE_FOR_mips_cmpu_le_qb, CODE_FOR_mips_pick_qb}, > + {CODE_FOR_mips_cmp_eq_ph, CODE_FOR_mips_pick_ph}, > + {CODE_FOR_mips_cmp_lt_ph, CODE_FOR_mips_pick_ph}, > + {CODE_FOR_mips_cmp_le_ph, CODE_FOR_mips_pick_ph}, > + {CODE_FOR_mips_wrdsp, CODE_FOR_mips_insv} > +}; > + > +int > +mips_dspalu_bypass_p (rtx out_insn, rtx in_insn) > +{ > + int i; > + int num_bypass = (sizeof (dspalu_bypass_table) > + / (2 * sizeof (enum insn_code))); this changed to ARRAY_SIZE (dspalu_bypass_table); Richard