On Fri, Sep 30, 2011 at 10:03 AM, Revital Eres <revital.e...@linaro.org> wrote: > Hello, > >> This >> + /* Skip instructions that do not set a register. */ >> + if (set && !REG_P (SET_DEST (set))) >> + continue; >> is ok. Can you also prevent !set insns from having reg_moves? (To be updated >> once auto_inc insns will be supported, if they'll deserve reg_moves too.) > > I added a check to verify that no reg-moves are created for !set instructions. > > Currently re-testing on ppc64-redhat-linux (bootstrap and regtest) and > arm-linux-gnueabi (bootstrap c). > > OK to commit once tesing completes? >
OK. later case >> latter case. Ayal. > Thanks, > Revital > > gcc/ > * modulo-sched.c (generate_reg_moves): Skip instructions that > do not set a register and verify no regmoves are created for > !single_set instructions. > > > testsuite/ > * gcc.dg/sms-10.c: New file. >
