[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-28 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #19 from Tom de Vries --- Filed spinoff PR80552 - "Make consecutive relative line numbers more maintainable" to capture discussion related to 'continue' line and dg-{begin,end}-same-line-output.

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-28 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #18 from Tom de Vries --- (In reply to Thomas Schwinge from comment #9) > You could further optimize the script to omit "." locations: if the "dg-*" > directive actually is placed on the appropriate line already. I did this in a

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-28 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 Tom de Vries changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-21 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #16 from Tom de Vries --- Created attachment 41243 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41243=edit tentative patch implementing dg-save-linenr

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-19 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #15 from Tom de Vries --- Author: vries Date: Wed Apr 19 06:55:33 2017 New Revision: 246988 URL: https://gcc.gnu.org/viewcvs?rev=246988=gcc=rev Log: Use relative line number for subsequent dg directives 2017-04-19 Tom de Vries

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-18 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #14 from Tom de Vries --- Created attachment 41217 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41217=edit Result from running updated script Untested for now.

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-18 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #13 from Tom de Vries --- Created attachment 41216 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41216=edit updated script patch This updated patch rewrites a line from absolute to relative linenumber if: - the relative line

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-18 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #12 from Tom de Vries --- (In reply to Mike Stump from comment #6) > The . and .-1, .+1, .-2 forms are fine. The .-62 forms are as problematic > as the original I suspect. I think we should exclude any number greater > than some

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-04-18 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 Bug 80221 depends on bug 80220, which changed state. Bug 80220 Summary: relative line numbers don't work when put between braces https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80220 What|Removed |Added

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-28 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #11 from Martin Sebor --- That would work too. Yet another possibility is to surround all the dg-message directives that apply to the same line in a pair of new directives (analogously to dg-{begin,end}-multiline-output):

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-28 Thread tschwinge at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #10 from Thomas Schwinge --- (In reply to Martin Sebor from comment #7) > A more robust solution that wouldn't have to subject to a limit would be to > add an annotation to dg-{error,message,warning} to indicate that the next >

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-28 Thread tschwinge at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 Thomas Schwinge changed: What|Removed |Added CC||tschwinge at gcc dot gnu.org ---

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-28 Thread tschwinge at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #9 from Thomas Schwinge --- You could further optimize the script to omit "." locations: if the "dg-*" directive actually is placed on the appropriate line already.

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-27 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 Martin Sebor changed: What|Removed |Added CC||msebor at gcc dot gnu.org --- Comment #7

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-27 Thread mikestump at comcast dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #6 from Mike Stump --- Comment on attachment 41059 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41059 Result from running script The . and .-1, .+1, .-2 forms are fine. The .-62 forms are as problematic as the original I

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-27 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #5 from Tom de Vries --- (In reply to Tom de Vries from comment #3) > Created attachment 41058 [details] > tested patch tested in combination with fix for PR80220.

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-27 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #4 from Tom de Vries --- Created attachment 41059 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41059=edit Result from running script

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-27 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 Tom de Vries changed: What|Removed |Added Attachment #41057|0 |1 is obsolete|

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-27 Thread mikestump at comcast dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 Mike Stump changed: What|Removed |Added CC||mikestump at comcast dot net --- Comment

[Bug testsuite/80221] Contrib script to rewrite testcase from absolute to relative line numbers

2017-03-27 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80221 --- Comment #1 from Tom de Vries --- Created attachment 41057 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41057=edit tentative patch This script tries to rewrite all tests, but that's not entirely supported yet. Known error causes: -