]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
[PR middle-end/117263] Avoid unused-but-set warning in genautomata
authorDario Gjorgjevski <dario.gjorgjevski@gmail.com>
Sun, 9 Feb 2025 16:16:31 +0000 (09:16 -0700)
committerJeff Law <jlaw@ventanamicro.com>
Sun, 9 Feb 2025 16:16:31 +0000 (09:16 -0700)
This is a trivial bug where a user wanted to define NDEBUG when building
genautomata, presumably trying to debug its behavior.  This resulted in a
unused-but-set warning which caused the build to fail.

Dario included the trivial fixes in the PR which I put through the usual
bootstrap & regression test as well as compiling genautomata with NDEBUG.

Pushing to the trunk.

PR middle-end/117263
gcc/
* genautomata.cc (output_statistics): Avoid set but unnused warnings
when compiling with NDEBUG.

gcc/genautomata.cc

index 69f856d141cd75a3db1f25da67547d5792c39a97..4059a229f961d7a7838ba4ef03464c42afc9d036 100644 (file)
@@ -9088,8 +9088,8 @@ static void
 output_statistics (FILE *f)
 {
   automaton_t automaton;
-  int states_num;
 #ifndef NDEBUG
+  int states_num;
   int transition_comb_vect_els = 0;
   int transition_full_vect_els = 0;
   int min_issue_delay_vect_els = 0;
@@ -9106,13 +9106,17 @@ output_statistics (FILE *f)
               automaton->NDFA_states_num, automaton->NDFA_arcs_num);
       fprintf (f, "    %5d DFA states,           %5d DFA arcs\n",
               automaton->DFA_states_num, automaton->DFA_arcs_num);
+#ifndef NDEBUG
       states_num = automaton->DFA_states_num;
+#endif
       if (!no_minimization_flag)
        {
          fprintf (f, "    %5d minimal DFA states,   %5d minimal DFA arcs\n",
                   automaton->minimal_DFA_states_num,
                   automaton->minimal_DFA_arcs_num);
+#ifndef NDEBUG
          states_num = automaton->minimal_DFA_states_num;
+#endif
        }
       fprintf (f, "    %5d all insns      %5d insn equivalence classes\n",
               description->insns_num, automaton->insn_equiv_classes_num);