Re: [PATCH] A tweak for debug_ops_vector

2015-10-08 Thread Bernd Schmidt 

On 10/08/2015 10:57 AM, Marek Polacek wrote:

I've been playing with reassoc recently and what bothers me is that
dump_ops_vector doesn't print a newline after each element, so it prints
e.g. "Op 0 -> rank: 5, tree: a_1(D)Op 1 -> rank: 4, tree: c_3(D)" which
isn't really all that readable to me.

Bootstrapped/regtested on x86_64-linux, ok for trunk?

2015-10-08  Marek Polacek  

* tree-ssa-reassoc.c (dump_ops_vector): Print newline after each
ops element.


Ok.


Bernd

[PATCH] A tweak for debug_ops_vector

2015-10-08 Thread Marek Polacek 
I've been playing with reassoc recently and what bothers me is that
dump_ops_vector doesn't print a newline after each element, so it prints
e.g. "Op 0 -> rank: 5, tree: a_1(D)Op 1 -> rank: 4, tree: c_3(D)" which
isn't really all that readable to me.

Bootstrapped/regtested on x86_64-linux, ok for trunk?

2015-10-08  Marek Polacek  

* tree-ssa-reassoc.c (dump_ops_vector): Print newline after each
ops element.

diff --git gcc/tree-ssa-reassoc.c gcc/tree-ssa-reassoc.c
index 5efee21..879722e 100644
--- gcc/tree-ssa-reassoc.c
+++ gcc/tree-ssa-reassoc.c
@@ -4930,6 +4930,7 @@ dump_ops_vector (FILE *file, vec ops)
 {
   fprintf (file, "Op %d -> rank: %d, tree: ", i, oe->rank);
   print_generic_expr (file, oe->op, 0);
+  fprintf (file, "\n");
 }
 }
 

Marek