On Wed, Jul 3, 2019 at 2:39 PM Martin Liška <mli...@suse.cz> wrote:
>
> On 7/3/19 1:48 PM, Richard Biener wrote:
> > On Wed, Jul 3, 2019 at 12:39 PM Martin Liška <mli...@suse.cz> wrote:
> >>
> >> Hi.
> >>
> >> The patch is about dbgcnt support for match in GIMPLE and GENERIC.
> >> It's handy for isolation of a problem.
> >>
> >> Ready to be installed after it survives tests?
> >
> > Hmm, I think we only want this on (simplify...), not (match...), no?
> > Thus simply move the code into the preceeding if() body.
>
> I've done that with s->kind == simplify::SIMPLIFY. Would it be fine?
>
> >
> > I'd also merge gimple and generic_match counters since passes
> > happen to use both.
>
> Works for me.

OK.

Thanks,
Richard.

> Martin
>
> >
> > OK with that changes.
> > Richard.
> >
> >> Thanks,
> >> Martin
> >>
> >> gcc/ChangeLog:
> >>
> >> 2019-07-03  Martin Liska  <mli...@suse.cz>
> >>
> >>         * dbgcnt.def (DEBUG_COUNTER): Add gimple_match and
> >>         generic_match.
> >>         * genmatch.c (dt_simplify::gen_1): Generate dbgcnt
> >>         condition.
> >>         * generic-match-head.c: Include dbgcnt.h.
> >>         * gimple-match-head.c: Likewise.
> >> ---
> >>  gcc/dbgcnt.def           | 2 ++
> >>  gcc/generic-match-head.c | 2 +-
> >>  gcc/genmatch.c           | 5 +++++
> >>  gcc/gimple-match-head.c  | 2 +-
> >>  4 files changed, 9 insertions(+), 2 deletions(-)
> >>
> >>
>

Reply via email to