[PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-20 Thread Arsen Arsenović via Gcc-patches
This script is a helper used to generate dg-output lines from an existing program output conveniently. It takes care of escaping Tcl and ARE stuff. contrib/ChangeLog: * dg-out-generator.pl: New file. --- contrib/dg-out-generator.pl | 79 + 1 file chan

[PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-22 Thread Arsen Arsenović via Gcc-patches
This script is a helper used to generate dg-output lines from an existing program output conveniently. It takes care of escaping Tcl and ARE stuff. contrib/ChangeLog: * dg-out-generator.pl: New file. --- I updated this file to include the proper copyright header, after dkm notified me th

Re: [PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-20 Thread Jonathan Wakely via Gcc-patches
On Tue, 20 Dec 2022 at 10:49, Arsen Arsenović wrote: > > This script is a helper used to generate dg-output lines from an existing > program output conveniently. It takes care of escaping Tcl and ARE stuff. > > contrib/ChangeLog: > > * dg-out-generator.pl: New file. > --- > contrib/dg-ou

Re: [PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-22 Thread Jason Merrill via Gcc-patches
On 12/22/22 06:03, Arsen Arsenović wrote: This script is a helper used to generate dg-output lines from an existing program output conveniently. It takes care of escaping Tcl and ARE stuff. contrib/ChangeLog: * dg-out-generator.pl: New file. --- I updated this file to include the prope

Re: [PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-22 Thread Arsen Arsenović via Gcc-patches
Hi, Jason Merrill writes: >> +# Newlines should be more tolerant. >> +s/\n$/(\\n|\\r\\n|\\r)*/; > > Isn't specifically handling \\r\\n redundant with the * operator? To the extent of my knowledge, yes; I left that in since the original tests I was replacing with this script also used thi

Re: [PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-22 Thread Jason Merrill via Gcc-patches
On 12/22/22 16:56, Arsen Arsenović wrote: Hi, Jason Merrill writes: +# Newlines should be more tolerant. +s/\n$/(\\n|\\r\\n|\\r)*/; Isn't specifically handling \\r\\n redundant with the * operator? To the extent of my knowledge, yes; I left that in since the original tests I was re

Re: [PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-22 Thread Arsen Arsenović via Gcc-patches
Jason Merrill writes: > Aha, I wonder why the original tests have the terminal *? Testcases elsewhere > in the testsuite that check for (\n|\r\n|\r) don't use *. I think I'll drop > the * from both the tests and the script. > > Jason Yep, that sounds reasonable. I'm not sure why, the origina