]> git.ipfire.org Git - thirdparty/gcc.git/commit
Fix pretty printers regexp for GDB output
authorChristophe Lyon <christophe.lyon@linaro.org>
Thu, 25 Jan 2024 15:43:56 +0000 (15:43 +0000)
committerChristophe Lyon <christophe.lyon@linaro.org>
Tue, 30 Apr 2024 08:59:08 +0000 (08:59 +0000)
commit6d4593a178b49cab205d81cdf36f52e12eabbc6d
tree3cc9d6a9c1d849d45c2aa9dea83ec26f793244f1
parent04ef92a62af3a815b86a2037267cd4e747ae225c
Fix pretty printers regexp for GDB output

GDB emits end of lines as \r\n, we currently match any >0 number of
either \n or \r, possibly leading to mismatches under racy conditions.

I noticed this while running the GCC testsuite using the equivalent of
GDB's READ1 feature [1] which helps detecting bufferization issues.

We try to match
\n$1 = empty std::tuple\r

against {^(type|\$([0-9]+)) = ([^\n\r]*)[\n\r]+} which fails because
of the leading \n (which was left in the buffer after the previous
"skipping" pattern matched the preceding \r).

This patch accepts any number of leading \n and/or \r in the "got" clause.

Also take this opportunity to quote \r and \r in the logs, to make
debugging such issues easier.

Tested on aarch64-linux-gnu.

[1] https//github.com/bminor/binutils-gdb/blob/master/gdb/testsuite/README#L269

2024-01-24  Christophe Lyon  <christophe.lyon@linaro.org>

libstdc++-v3/
* testsuite/lib/gdb-test.exp (gdb-test): Fix regexp.  Quote
newlines in logs.
libstdc++-v3/testsuite/lib/gdb-test.exp