Comments

Hello,
> OK, with the following comments:>> Make sure reg_moves are generated for the correct (result, not addr)> register, in generate_reg_moves().>> "been">>"being" (multiple appearances).>> Add a note that autoinc_var_is_used_p (rtx def_insn, rtx use_insn)> doesn't need to consider the specific address register; no reg_moves> will be allowed for any life range defined by def_insn and used by> use_insn, if use_insn uses an address register auto-inc'ed by> def_insn.
Attached is the version of the patch which addresses your comments.
Currently re-testing on ppc64-redhat-linux (bootstrap and regtest) and
arm-linux-gnueabi (bootstrap c).
I'll commit it once tesing completes if there if no further changes required.
Thanks,
Revital
* ddg.c (autoinc_var_is_used_p): New function.
(create_ddg_dep_from_intra_loop_link,
add_cross_iteration_register_deps): Call it.
* ddg.h (autoinc_var_is_used_p): Declare.
* modulo-sched.c (generate_reg_moves): Call autoinc_var_is_used_p.
(sms_schedule): Handle instructions with REG_INC.