diff mbox

A tweak for debug_ops_vector

Message ID 20151008085722.GZ6184@redhat.com
State New
Headers show

Commit Message

Marek Polacek Oct. 8, 2015, 8:57 a.m. UTC 
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  <polacek@redhat.com>

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


	Marek

Comments

Bernd Schmidt Oct. 8, 2015, 9:48 a.m. UTC | #1 
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  <polacek@redhat.com>
>
> 	* tree-ssa-reassoc.c (dump_ops_vector): Print newline after each
> 	ops element.

Ok.


Bernd
diff mbox

Patch

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<operand_entry *> ops)
     {
       fprintf (file, "Op %d -> rank: %d, tree: ", i, oe->rank);
       print_generic_expr (file, oe->op, 0);
+      fprintf (file, "\n");
     }
 }