https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109927#c21 From 2d83fafb3f263544be09bf7dbde036e4a84f6d7c Mon Sep 17 00:00:00 2001 From: Tamar Christina Date: Fri, 5 May 2023 13:36:43 +0100 Subject: [PATCH 03/15] match.pd: CSE the dump output check. This is a small improvement in QoL codegen for match.pd to save time not re-evaluating the condition for printing debug information in every function. There is a small but consistent runtime and compile time win here. The runtime win comes from not having to do the condition over again, and on Arm plaforms we now use the new test-and-branch support for booleans to only have a single instruction here. gcc/ChangeLog: PR bootstrap/84402 * genmatch.cc (decision_tree::gen, write_predicate): Generate new debug_dump var. (dt_simplify::gen_1): Use it. (cherry picked from commit c0ce29bc1ce329001b6c02bb3d34bcbb086e1b72) --- gcc/genmatch.cc | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/gcc/genmatch.cc b/gcc/genmatch.cc index 45e49c790b6d..a17ef8a23ed5 100644 --- a/gcc/genmatch.cc +++ b/gcc/genmatch.cc @@ -3433,7 +3433,7 @@ dt_simplify::gen_1 (FILE *f, int indent, bool gimple, operand *result) needs_label = true; } - fprintf_indent (f, indent, "if (UNLIKELY (dump_file && (dump_flags & TDF_FOLDING))) " + fprintf_indent (f, indent, "if (UNLIKELY (debug_dump)) " "fprintf (dump_file, \"%s ", s->kind == simplify::SIMPLIFY ? "Applying pattern" : "Matching expression"); @@ -3894,6 +3894,8 @@ decision_tree::gen (FILE *f, bool gimple) } fprintf (f, ")\n{\n"); + fprintf_indent (f, 2, "const bool debug_dump = " + "dump_file && (dump_flags & TDF_FOLDING);\n"); s->s->gen_1 (f, 2, gimple, s->s->s->result); if (gimple) fprintf (f, " return false;\n"); @@ -3939,6 +3941,8 @@ decision_tree::gen (FILE *f, bool gimple) fprintf (f, ", tree _p%d", i); fprintf (f, ")\n"); fprintf (f, "{\n"); + fprintf_indent (f, 2, "const bool debug_dump = " + "dump_file && (dump_flags & TDF_FOLDING);\n"); dop->gen_kids (f, 2, gimple, 0); if (gimple) fprintf (f, " return false;\n"); @@ -4048,6 +4052,8 @@ write_predicate (FILE *f, predicate_id *p, decision_tree &dt, bool gimple) gimple ? ", tree (*valueize)(tree) ATTRIBUTE_UNUSED" : ""); /* Conveniently make 'type' available. */ fprintf_indent (f, 2, "const tree type = TREE_TYPE (t);\n"); + fprintf_indent (f, 2, "const bool debug_dump = " + "dump_file && (dump_flags & TDF_FOLDING);\n"); if (!gimple) fprintf_indent (f, 2, "if (TREE_SIDE_EFFECTS (t)) return false;\n"); -- 2.44.0