Hello,

On 26.10.2011 21:26, Alexander Monakov wrote:
(Note, this is a prerequisite for predication support but also is an
  improvement on its own.  This patch can be installed separately.)

...

diff --git a/gcc/sel-sched.c b/gcc/sel-sched.c
index f11faca..91fb0fe 100644
--- a/gcc/sel-sched.c
+++ b/gcc/sel-sched.c
@@ -5234,6 +5234,7 @@ move_exprs_to_boundary (bnd_t bnd, expr_t expr_vliw,

    b = move_op (BND_TO (bnd), expr_seq, expr_vliw,
                 get_dest_from_orig_ops (expr_seq), c_expr,&should_move);
+  free_history_vect (&EXPR_HISTORY_OF_CHANGES (c_expr));
Why do we need this? Upon returning from move_exprs_to_boundary (in schedule_expr_on_boundary), we immediately call clear_expr on c_expr which in turn calls free_history_vect. So the history vector would be cleared anyways, no? In this case we don't have to export free_history_vect, too.

Otherwise OK.

Thanks,
Andrey

Reply via email to