Re: [PATCH] proper dataflow block visitation order

["Steven Bosscher" <stevenb.gcc@gmail.com>]

xf. http://gcc.gnu.org/ml/gcc-patches/2008-02/msg00269.html

Seongbae wrote:
> +
> +  /* Make them reverse postorder.  */
> +  reverse_array (this_df->n_blocks, this_df->postorder);
> +  reverse_array (this_df->n_blocks, this_df->postorder_inverted);
> +}

Could you also change the names, then, to df->reverse_postorder and
df->reverse_postorder_inverted? This is going to confuse the heck out
of people who want to use these for their own purposes.

Maybe you can use pre_and_rev_post_order_compute() for df->postorder,
instead of computing postorder and reverting it with reverse_array().

And actually, what is wrong with just walking these arrays from the
last element to the first, instead of using reverse_array()?

> -  gcc_assert (dir != DF_NONE);
> -

Why are you removing this assert?

Gr.
Steven