diff mbox

[match-and-simplify] Remove printing "for expression"

Message ID CAAgBjMk4GAnnPof_=0zL8ePNY61KL+gi7E3934AVRJbUDmEM7w@mail.gmail.com
State New
Headers show

Commit Message

Prathamesh Kulkarni Dec. 21, 2014, 9:51 p.m. UTC 
Hi,
I removed printing "for expression:" from print_matches. I think it
is out of place tvim here and we call print_matches after lowering.
OK to commit ?

Thanks,
Prathamesh
2014-12-12  Prathamesh Kulkarni  <prathamesh.kulkarni@linaro.org>

	* genmatch.c (print_matches): Change type of first parameter to operand *.
	Remove call to fprintf to print "for expression: ".
	(main): Adjust call to print_matches.

Comments

Prathamesh Kulkarni Dec. 21, 2014, 9:52 p.m. UTC | #1 
On 22 December 2014 at 03:21, Prathamesh Kulkarni
<prathamesh.kulkarni@linaro.org> wrote:
> Hi,
> I removed printing "for expression:" from print_matches. I think it
> is out of place tvim here and we call print_matches after lowering.
> OK to commit ?
s/"tvim here"/there.
>
> Thanks,
> Prathamesh
diff mbox

Patch

Index: gcc/genmatch.c
===================================================================
--- gcc/genmatch.c	(revision 218996)
+++ gcc/genmatch.c	(working copy)
@@ -645,10 +645,9 @@ 
 }
 
 DEBUG_FUNCTION void
-print_matches (struct simplify *s, FILE *f = stderr)
+print_matches (operand *o, FILE *f = stderr)
 {
-  fprintf (f, "for expression: ");
-  print_operand (s->match, f);
+  print_operand (o, f);
   putc ('\n', f);
 }
 
@@ -3672,7 +3671,7 @@ 
 
       if (verbose)
 	for (unsigned i = 0; i < pred->matchers.length (); ++i)
-	  print_matches (pred->matchers[i]);
+	  print_matches (pred->matchers[i]->match);
 
       decision_tree dt;
       for (unsigned i = 0; i < pred->matchers.length (); ++i)
@@ -3689,7 +3688,7 @@ 
 
   if (verbose)
     for (unsigned i = 0; i < p.simplifiers.length (); ++i)
-      print_matches (p.simplifiers[i]);
+      print_matches (p.simplifiers[i]->match);
 
   decision_tree dt;
   for (unsigned i = 0; i < p.simplifiers.length (); ++i)