[PATCH] tree level if-conversion for vectorizer

[Zdenek Dvorak]

Hello,

why do you need this function?

> + basic_block *
> + get_loop_body_in_bfs_order (const struct loop *loop)

I see two uses:

1) In handle_sibling_pattern. get_loop_body_in_dom_order
   seems more appropriate here.
2) In get_loop_body_in_if_conv_order;  what you are trying
   to get here (as far as I understand) is loop body in
   topological sort ordering; doing it using
   get_loop_body_in_bfs_order is cumbersome.

> +   compute_immediate_uses (TDFA_USE_OPS, NULL);
> +   compute_immediate_uses (TDFA_USE_VOPS, NULL);

Why not
  compute_immediate_uses (TDFA_USE_OPS | TDFA_USE_VOPS, NULL)?

Zdenek