Commit cbd8652b by David Malcolm Committed by David Malcolm

multiline.exp: complain about mismatched dg-{begin|end}-multiline-output

Mismatched dg-{begin|end}-multiline-output directives are currently
silently ignored, leading to difficult-to-diagnose test failures
involving excess output.

This patch makes multiline.exp complain about them.

gcc/testsuite/ChangeLog:
	* lib/multiline.exp (dg-begin-multiline-output): Issue an error if
	there hasn't been a dg-end-multiline-output since the last
	dg-begin-multiline-output.
	(dg-end-multiline-output): Issue an error if there hasn't been a
	dg-begin-multiline-output.  Reset _multiline_last_beginning_line
	as soon possible.  Rename "line" to "last_line".

From-SVN: r265046
parent cfef4c32
2018-10-11 David Malcolm <dmalcolm@redhat.com>
* lib/multiline.exp (dg-begin-multiline-output): Issue an error if
there hasn't been a dg-end-multiline-output since the last
dg-begin-multiline-output.
(dg-end-multiline-output): Issue an error if there hasn't been a
dg-begin-multiline-output. Reset _multiline_last_beginning_line
as soon possible. Rename "line" to "last_line".
2018-10-11 Andrew Stubbs <ams@codesourcery.com> 2018-10-11 Andrew Stubbs <ams@codesourcery.com>
* selftests/repeat.rtl: New file. * selftests/repeat.rtl: New file.
......
...@@ -72,6 +72,14 @@ proc dg-begin-multiline-output { args } { ...@@ -72,6 +72,14 @@ proc dg-begin-multiline-output { args } {
global _multiline_last_beginning_line global _multiline_last_beginning_line
verbose "dg-begin-multiline-output: args: $args" 3 verbose "dg-begin-multiline-output: args: $args" 3
set line [expr [lindex $args 0] + 1] set line [expr [lindex $args 0] + 1]
# Complain if there hasn't been a dg-end-multiline-output
# since the last dg-begin-multiline-output
if { $_multiline_last_beginning_line != -1 } {
set last_directive_line [expr $_multiline_last_beginning_line - 1]
error "$last_directive_line: unterminated dg-begin-multiline-output"
}
set _multiline_last_beginning_line $line set _multiline_last_beginning_line $line
} }
...@@ -84,8 +92,17 @@ proc dg-begin-multiline-output { args } { ...@@ -84,8 +92,17 @@ proc dg-begin-multiline-output { args } {
proc dg-end-multiline-output { args } { proc dg-end-multiline-output { args } {
global _multiline_last_beginning_line global _multiline_last_beginning_line
verbose "dg-end-multiline-output: args: $args" 3 verbose "dg-end-multiline-output: args: $args" 3
set line [expr [lindex $args 0] - 1] set first_line $_multiline_last_beginning_line
verbose "multiline output lines: $_multiline_last_beginning_line-$line" 3
# Complain if there hasn't been a dg-begin-multiline-output
if { $first_line == -1 } {
error "[lindex $args 0]: dg-end-multiline-output without dg-begin-multiline-output"
return
}
set _multiline_last_beginning_line -1
set last_line [expr [lindex $args 0] - 1]
verbose "multiline output lines: $first_line-$last_line" 3
if { [llength $args] > 3 } { if { [llength $args] > 3 } {
error "[lindex $args 0]: too many arguments" error "[lindex $args 0]: too many arguments"
...@@ -109,16 +126,14 @@ proc dg-end-multiline-output { args } { ...@@ -109,16 +126,14 @@ proc dg-end-multiline-output { args } {
# "prog" now contains the filename # "prog" now contains the filename
# Load it and split it into lines # Load it and split it into lines
set lines [_get_lines $prog $_multiline_last_beginning_line $line] set lines [_get_lines $prog $first_line $last_line]
verbose "lines: $lines" 3 verbose "lines: $lines" 3
# Create an entry of the form: first-line, last-line, lines, maybe_x # Create an entry of the form: first-line, last-line, lines, maybe_x
set entry [list $_multiline_last_beginning_line $line $lines $maybe_x] set entry [list $first_line $last_line $lines $maybe_x]
global multiline_expected_outputs global multiline_expected_outputs
lappend multiline_expected_outputs $entry lappend multiline_expected_outputs $entry
verbose "within dg-end-multiline-output: multiline_expected_outputs: $multiline_expected_outputs" 3 verbose "within dg-end-multiline-output: multiline_expected_outputs: $multiline_expected_outputs" 3
set _multiline_last_beginning_line -1
} }
# Hook to be called by prune.exp's prune_gcc_output to # Hook to be called by prune.exp's prune_gcc_output to
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment