Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom de Vries <tdevries@suse.de>
To: gdb-patches@sourceware.org
Subject: [PATCH 0/5] [gdb/testsuite] Use valnum_re more often
Date: Wed, 28 Jan 2026 13:06:21 +0100	[thread overview]
Message-ID: <20260128120626.1032047-1-tdevries@suse.de> (raw)

In the gdb testsuite we use the valnum_re regexp to match the '$1' in:
...
(gdb) print 1
$1 = 1
(gdb)
...

This series consists of valnum_re related patches, mostly using it more often.

I started out writing a patch series using decimal and hex more often (and
other related improvements), but put that on hold once I realized that:
- I also wanted to use valnum_re more often, and
- it doesn't make sense to first rewrite "\\\$\[0-9\]+" into "\\\$$decimal"
  and then into "$valnum_re".

Tested on x86_64-linux.  I've reviewed the changes in gdb.arch.

Tom de Vries (5):
  [gdb/testsuite] Remove superfluous definition of valnum_re in
    gdb.base/commands.exp
  [gdb/testsuite] Use valnum_re more often (part 1)
  [gdb/testsuite] Use valnum_re more often (part 2)
  [gdb/testsuite] Use valnum_re more often (part 3)
  [gdb/testsuite] Use valnum_re more often (part 4)

 gdb/testsuite/gdb.ada/array_return.exp        |   6 +-
 gdb/testsuite/gdb.ada/formatted_ref.exp       |  14 +-
 gdb/testsuite/gdb.ada/operator_call.exp       |   2 +-
 gdb/testsuite/gdb.arch/aarch64-gcs-core.exp   |   2 +-
 gdb/testsuite/gdb.arch/aarch64-mte.exp        |   2 +-
 .../gdb.arch/amd64-shadow-stack-corefile.exp  |   2 +-
 gdb/testsuite/gdb.arch/iwmmxt-regs.exp        |  12 +-
 gdb/testsuite/gdb.arch/mips-fcr.exp           |   4 +-
 gdb/testsuite/gdb.arch/mips16-thunks.exp      |   2 +-
 gdb/testsuite/gdb.arch/powerpc-d128-regs.exp  |   2 +-
 .../gdb.arch/powerpc-plxv-nonrel.exp          |   2 +-
 gdb/testsuite/gdb.arch/s390-tdbregs.exp       |   6 +-
 gdb/testsuite/gdb.base/args.exp               |   4 +-
 gdb/testsuite/gdb.base/bitfields.exp          |  12 +-
 gdb/testsuite/gdb.base/bitshift.exp           |   4 +-
 .../gdb.base/bp-cmds-sourced-script.exp       |   2 +-
 .../gdb.base/break-main-file-remove-fail.exp  |   2 +-
 gdb/testsuite/gdb.base/charset.exp            |   2 +-
 gdb/testsuite/gdb.base/code_elim.exp          |   2 +-
 gdb/testsuite/gdb.base/commands.exp           |   3 -
 gdb/testsuite/gdb.base/complex.exp            |   4 +-
 gdb/testsuite/gdb.base/coredump-filter.exp    |   2 +-
 gdb/testsuite/gdb.base/corefile.exp           |   8 +-
 gdb/testsuite/gdb.base/debug-expr.exp         |   4 +-
 gdb/testsuite/gdb.base/dump.exp               |   4 +-
 gdb/testsuite/gdb.base/exprs.exp              | 222 +++++++++---------
 gdb/testsuite/gdb.base/finish.exp             |   2 +-
 gdb/testsuite/gdb.base/frameapply.exp         |  38 +--
 gdb/testsuite/gdb.base/func-ptrs.exp          |   6 +-
 gdb/testsuite/gdb.base/gdb1090.exp            |   4 +-
 gdb/testsuite/gdb.base/gnu_vector.exp         | 116 ++++-----
 gdb/testsuite/gdb.base/infnan.exp             |   4 +-
 gdb/testsuite/gdb.base/interact.exp           |   2 +-
 .../gdb.base/multi-line-starts-subshell.exp   |   2 +-
 gdb/testsuite/gdb.base/new-ui.exp             |   2 +-
 .../gdb.base/non-lazy-array-index.exp         |   2 +-
 gdb/testsuite/gdb.base/offsets.exp            |   4 +-
 gdb/testsuite/gdb.base/overlays.exp           |   2 +-
 gdb/testsuite/gdb.base/prologue.exp           |   4 +-
 gdb/testsuite/gdb.base/ptr-typedef.exp        |   4 +-
 gdb/testsuite/gdb.base/randomize.exp          |   2 +-
 gdb/testsuite/gdb.base/range-stepping.exp     |   4 +-
 gdb/testsuite/gdb.base/readline.exp           |   8 +-
 gdb/testsuite/gdb.base/relocate.exp           |   8 +-
 gdb/testsuite/gdb.base/scope.exp              | 112 ++++-----
 gdb/testsuite/gdb.base/set-cwd.exp            |  10 +-
 .../gdb.base/share-env-with-gdbserver.exp     |   6 +-
 gdb/testsuite/gdb.base/shell.exp              |   4 +-
 gdb/testsuite/gdb.base/signals.exp            |   2 +-
 gdb/testsuite/gdb.base/solib-disc.exp         |   8 +-
 gdb/testsuite/gdb.base/startup-with-shell.exp |   2 +-
 gdb/testsuite/gdb.base/structs3.exp           |   2 +-
 gdb/testsuite/gdb.base/style.exp              |   4 +-
 gdb/testsuite/gdb.cp/anon-union.exp           |  30 +--
 gdb/testsuite/gdb.cp/bs15503.exp              |   8 +-
 gdb/testsuite/gdb.cp/call-c.exp               |   6 +-
 gdb/testsuite/gdb.cp/classes.exp              |  26 +-
 gdb/testsuite/gdb.cp/converts.exp             |   2 +-
 gdb/testsuite/gdb.cp/cp-relocate.exp          |   2 +-
 gdb/testsuite/gdb.cp/ctti.exp                 |  30 +--
 gdb/testsuite/gdb.cp/derivation.exp           |  36 +--
 gdb/testsuite/gdb.cp/formatted-ref.exp        |  18 +-
 gdb/testsuite/gdb.cp/inherit.exp              |   2 +-
 gdb/testsuite/gdb.cp/m-data.exp               |  32 +--
 gdb/testsuite/gdb.cp/m-static.exp             |  34 +--
 gdb/testsuite/gdb.cp/member-ptr.exp           |   2 +-
 gdb/testsuite/gdb.cp/method-ref-return.exp    |  12 +-
 gdb/testsuite/gdb.cp/method.exp               |  18 +-
 gdb/testsuite/gdb.cp/misc.exp                 |  42 ++--
 gdb/testsuite/gdb.cp/namespace.exp            |  58 ++---
 gdb/testsuite/gdb.cp/nsimport.exp             |   6 +-
 gdb/testsuite/gdb.cp/overload.exp             |  62 ++---
 gdb/testsuite/gdb.cp/pr-574.exp               |   2 +-
 gdb/testsuite/gdb.cp/printmethod.exp          |   4 +-
 gdb/testsuite/gdb.cp/rtti.exp                 |  14 +-
 gdb/testsuite/gdb.cp/static-print-quit.exp    |   2 +-
 gdb/testsuite/gdb.cp/templates.exp            |  10 +-
 gdb/testsuite/gdb.cp/userdef.exp              |  64 ++---
 gdb/testsuite/gdb.cp/virtfunc.exp             |  26 +-
 gdb/testsuite/gdb.cp/virtfunc2.exp            |  10 +-
 gdb/testsuite/gdb.fortran/debug-expr.exp      |   6 +-
 gdb/testsuite/gdb.fortran/derived-type.exp    |  16 +-
 gdb/testsuite/gdb.fortran/namelist.exp        |   2 +-
 gdb/testsuite/gdb.fortran/subarray.exp        |  24 +-
 .../gdb.mi/user-selected-context-sync.exp     |   2 +-
 gdb/testsuite/gdb.multi/tids.exp              |   2 +-
 gdb/testsuite/gdb.objc/basicclass.exp         |  10 +-
 .../gdb.opt/clobbered-registers-O2.exp        |   4 +-
 gdb/testsuite/gdb.opt/fortran-string.exp      |   2 +-
 gdb/testsuite/gdb.opt/inline-cmds.exp         |   2 +-
 gdb/testsuite/gdb.opt/inline-locals.exp       |   4 +-
 .../gdb.opt/static-optimized-out.exp          |   2 +-
 gdb/testsuite/gdb.server/ext-wrapper.exp      |   4 +-
 gdb/testsuite/gdb.server/inferior-args.exp    |   4 +-
 gdb/testsuite/gdb.server/wrapper.exp          |   2 +-
 .../gdb.threads/continue-pending-status.exp   |   2 +-
 .../gdb.threads/infcall-thread-announce.exp   |   6 +-
 gdb/testsuite/gdb.threads/pthreads.exp        |  12 +-
 .../gdb.threads/reconnect-signal.exp          |   4 +-
 gdb/testsuite/gdb.threads/schedlock.exp       |   2 +-
 .../gdb.threads/thread-bp-deleted.exp         |   2 +-
 gdb/testsuite/gdb.threads/threadapply.exp     |   4 +-
 gdb/testsuite/gdb.threads/tls-core.exp        |   2 +-
 gdb/testsuite/gdb.trace/collection.exp        | 100 ++++----
 .../gdb.trace/disconnected-tracing.exp        |  14 +-
 gdb/testsuite/gdb.trace/report.exp            |   4 +-
 gdb/testsuite/gdb.trace/stap-trace.exp        |   2 +-
 .../gdb.trace/unavailable-dwarf-piece.exp     |  18 +-
 gdb/testsuite/gdb.trace/unavailable.exp       |   8 +-
 gdb/testsuite/lib/gdb.exp                     |  12 +-
 gdb/testsuite/lib/mi-support.exp              |   2 +-
 111 files changed, 765 insertions(+), 768 deletions(-)


base-commit: c02c23175e47f29907a0a3af3f6e6a404bc99719
-- 
2.51.0


             reply	other threads:[~2026-01-28 12:09 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-28 12:06 Tom de Vries [this message]
2026-01-28 12:06 ` [PATCH 1/5] [gdb/testsuite] Remove superfluous definition of valnum_re in gdb.base/commands.exp Tom de Vries
2026-01-28 12:06 ` [PATCH 2/5] [gdb/testsuite] Use valnum_re more often (part 1) Tom de Vries
2026-01-28 12:06 ` [PATCH 3/5] [gdb/testsuite] Use valnum_re more often (part 2) Tom de Vries
2026-01-28 12:06 ` [PATCH 4/5] [gdb/testsuite] Use valnum_re more often (part 3) Tom de Vries
2026-01-28 12:06 ` [PATCH 5/5] [gdb/testsuite] Use valnum_re more often (part 4) Tom de Vries
2026-02-02 17:05 ` [PATCH 0/5] [gdb/testsuite] Use valnum_re more often Kevin Buettner
2026-02-03 11:14 ` Tom de Vries

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260128120626.1032047-1-tdevries@suse.de \
    --to=tdevries@suse.de \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox