Commit 413e50a2 by Revital Eres Committed by Revital Eres

SMS: Misc. fixes

From-SVN: r176970
parent f047e518
2011-07-31 Revital Eres <revital.eres@linaro.org>
* modulo-sched.c: Change comment.
(reset_sched_times): Fix print message.
(print_partial_schedule): Add print info.
2011-07-31 Tom de Vries <tom@codesourcery.com> 2011-07-31 Tom de Vries <tom@codesourcery.com>
PR middle-end/43513 PR middle-end/43513
......
...@@ -84,13 +84,14 @@ along with GCC; see the file COPYING3. If not see ...@@ -84,13 +84,14 @@ along with GCC; see the file COPYING3. If not see
II cycles (i.e. use register copies to prevent a def from overwriting II cycles (i.e. use register copies to prevent a def from overwriting
itself before reaching the use). itself before reaching the use).
SMS works with countable loops whose loop count can be easily SMS works with countable loops (1) whose control part can be easily
adjusted. This is because we peel a constant number of iterations decoupled from the rest of the loop and (2) whose loop count can
into a prologue and epilogue for which we want to avoid emitting be easily adjusted. This is because we peel a constant number of
the control part, and a kernel which is to iterate that constant iterations into a prologue and epilogue for which we want to avoid
number of iterations less than the original loop. So the control emitting the control part, and a kernel which is to iterate that
part should be a set of insns clearly identified and having its constant number of iterations less than the original loop. So the
own iv, not otherwise used in the loop (at-least for now), which control part should be a set of insns clearly identified and having
its own iv, not otherwise used in the loop (at-least for now), which
initializes a register before the loop to the number of iterations. initializes a register before the loop to the number of iterations.
Currently SMS relies on the do-loop pattern to recognize such loops, Currently SMS relies on the do-loop pattern to recognize such loops,
where (1) the control part comprises of all insns defining and/or where (1) the control part comprises of all insns defining and/or
...@@ -598,8 +599,8 @@ reset_sched_times (partial_schedule_ptr ps, int amount) ...@@ -598,8 +599,8 @@ reset_sched_times (partial_schedule_ptr ps, int amount)
/* Print the scheduling times after the rotation. */ /* Print the scheduling times after the rotation. */
fprintf (dump_file, "crr_insn->node=%d (insn id %d), " fprintf (dump_file, "crr_insn->node=%d (insn id %d), "
"crr_insn->cycle=%d, min_cycle=%d", crr_insn->node->cuid, "crr_insn->cycle=%d, min_cycle=%d", crr_insn->node->cuid,
INSN_UID (crr_insn->node->insn), SCHED_TIME (u), INSN_UID (crr_insn->node->insn), normalized_time,
normalized_time); new_min_cycle);
if (JUMP_P (crr_insn->node->insn)) if (JUMP_P (crr_insn->node->insn))
fprintf (dump_file, " (branch)"); fprintf (dump_file, " (branch)");
fprintf (dump_file, "\n"); fprintf (dump_file, "\n");
...@@ -2550,8 +2551,13 @@ print_partial_schedule (partial_schedule_ptr ps, FILE *dump) ...@@ -2550,8 +2551,13 @@ print_partial_schedule (partial_schedule_ptr ps, FILE *dump)
fprintf (dump, "\n[ROW %d ]: ", i); fprintf (dump, "\n[ROW %d ]: ", i);
while (ps_i) while (ps_i)
{ {
fprintf (dump, "%d, ", if (JUMP_P (ps_i->node->insn))
INSN_UID (ps_i->node->insn)); fprintf (dump, "%d (branch), ",
INSN_UID (ps_i->node->insn));
else
fprintf (dump, "%d, ",
INSN_UID (ps_i->node->insn));
ps_i = ps_i->next_in_row; ps_i = ps_i->next_in_row;
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment