Thanks for doing this. tbsaunde+...@tbsaunde.org writes: > diff --git a/gcc/genrecog.c b/gcc/genrecog.c > index a8e8c22..aa7f629 100644 > --- a/gcc/genrecog.c > +++ b/gcc/genrecog.c > @@ -5102,8 +5102,7 @@ print_subroutine (output_state *os, state *s, int > proc_id) > /* For now, the top-level "recog" takes a plain "rtx", and performs a > checked cast to "rtx_insn *" for use throughout the rest of the > function and the code it calls. */ > - const char *insn_param > - = proc_id > 0 ? "rtx_insn *insn" : "rtx uncast_insn"; > + const char *insn_param = "rtx_insn *insn";
The comment is no longer true after the patch. We might as well just get rid of the variable now that it's equal to a constant string of almost the same length as the variable name. Richard