This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: Weird behaviour with --target_board="unix{var1,var2}"
- From: Pedro Alves <pedro at palves dot net>
- To: Jonathan Wakely <jwakely at redhat dot com>
- Cc: gcc at gcc dot gnu dot org, Mike Stump <mrs at mrs dot kithrup dot com>, gcc-patches at gcc dot gnu dot org, libstdc++ at gcc dot gnu dot org
- Date: Tue, 23 Aug 2016 12:05:50 +0100
- Subject: Re: Weird behaviour with --target_board="unix{var1,var2}"
- Authentication-results: sourceware.org; auth=none
- References: <20160822144019.GT20016@redhat.com> <92ea5626-52cc-f515-7d11-25ef95c690a1@palves.net> <20160823080715.GY20016@redhat.com> <20160823095450.GB20016@redhat.com>
On 08/23/2016 10:54 AM, Jonathan Wakely wrote:
>> That's being set by prettyprinters.exp and xmethods.exp (so it's GDB's
>> fault! ;-)
:-)
> This seems to work. I'll do some more testing and commit later today.
LGTM.
Though IME, save/restoring globals in a constant source of trouble,
for occasionally someone adds an early return that inadvertently
skips the restore. Of course in gdb every test must be written
using custom .exp code, so we're more prone to being bitten. Still...
The way we solve that on gdb systematically is with convenience
wrappers that handle the save/restore. E.g. see:
$ grep "proc with_" *
gdb.exp:proc with_test_prefix { prefix body } {
gdb.exp:proc with_gdb_prompt { prompt body } {
gdb.exp:proc with_target_charset { target_charset body } {
gdb.exp:proc with_spawn_id { spawn_id body } {
gdb.exp:proc with_timeout_factor { factor body } {
In this particular case, I'd add a wrapper method to
libstdc++-v3/testsuite/lib/gdb-test.exp, such as:
# Like dg-runtest but keep the .exe around. dg-test has an option for
# this but there is no way to pass it through dg-runtest.
proc gdb-dg-runtest {args} {
global dg-interpreter-batch-mode
set saved-dg-interpreter-batch-mode ${dg-interpreter-batch-mode}
set dg-interpreter-batch-mode 1
eval dg-runtest $args
set dg-interpreter-batch-mode ${saved-dg-interpreter-batch-mode}
}
And then use gdb-dg-runtest instead of dg-runtest in
prettyprinters.exp and xmethods.exp.
(Maybe put even move more of the duplicate code around the
current dg-runtest calls to the wrapper, and then give the
wrapper named arguments.)