On Wed, Jan 15, 2025 at 2:30 AM Richard Biener <rguent...@suse.de> wrote:
>
> The following adds /* <num> */ to dbg_line_numbers so there's the chance
> to more easily lookup the ID of the match.pd line number used for
> dumping when you want to debug a speicific replacement. It also cuts
> the lines down to 10 entries.
>
> static int dbg_line_numbers[1267] = {
> /* 0 */ 161, 164, 173, 175, 178, 181, 183, 189, 197, 195,
> /* 10 */ 199, 201, 205, 923, 921, 2060, 2071, 2052, 2058, 2063,
> ...
>
> Queued for testing, will push.
>
> * genmatch.cc (define_dump_logs): Make reverse lookup in
> dbg_line_numbers easier by adding comments with start index
> and cutting number of elements per line to 10.
> ---
> gcc/genmatch.cc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/genmatch.cc b/gcc/genmatch.cc
> index 57fccafb1ef..b1315d6b4f4 100644
> --- a/gcc/genmatch.cc
> +++ b/gcc/genmatch.cc
> @@ -891,7 +891,7 @@ define_dump_logs (bool gimple, FILE *f)
> for (unsigned i = 0; i < dbg_line_numbers.length () - 1; i++)
> {
> if (i % 20 == 0)
That didn't change it to 10. Like you had said above.
But I did notice the patch which you committed had the change from 20 to 10.
https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=bea593f115bccffcb2570fc9cd642403193265d9
Thanks,
Andrew Pinski
> - fprintf (f, "\n\t");
> + fprintf (f, "\n\t/* %d */ ", i);
>
> fprintf (f, "%d, ", dbg_line_numbers[i]);
> }
> --
> 2.43.0
