Message ID | CAAgBjMk4GAnnPof_=0zL8ePNY61KL+gi7E3934AVRJbUDmEM7w@mail.gmail.com |
---|---|
State | New |
Headers | show |
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
On Sun, 21 Dec 2014, Prathamesh Kulkarni 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 ? Hum, it's now a very simple wrapper around print_operand - why not replace the two callers with its content? Thanks, Richard. > Thanks, > Prathamesh >
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)