]> git.ipfire.org Git - thirdparty/binutils-gdb.git/log
thirdparty/binutils-gdb.git
4 months agoReorder gnatmake arguments in inline-section-gc.exp
Tom Tromey [Mon, 3 Feb 2025 19:12:29 +0000 (12:12 -0700)] 
Reorder gnatmake arguments in inline-section-gc.exp

inline-section-gc.exp ends up passing "-lm" to gnatmake as an "marg"
-- meaning gnatmake should process it itself.  However, the gnat-llvm
gnatmake does not know what to do with this, so the test fails.

This patch rearranges the arguments so that the (implicit) trailing
-lm ends up being passed through to the linker.

4 months agodoc: sframe: Clarify FDE/FRE function/range start address fields
Jens Remus [Tue, 4 Feb 2025 14:13:24 +0000 (15:13 +0100)] 
doc: sframe: Clarify FDE/FRE function/range start address fields

The function start address in a SFrame FDE (sfde_func_start_address)
is encoded as a signed offset to the function start address from the
SFrame section.

The PC range start address in a SFrame FRE (sfre_start_address) is
encoded as an unsigned offset to the range from the function start
address.

Signed-off-by: Jens Remus <jremus@linux.ibm.com>
4 months agogas: Skip SFrame FDE if .cfi_val_offset specifies non-default offset
Jens Remus [Tue, 4 Feb 2025 14:13:24 +0000 (15:13 +0100)] 
gas: Skip SFrame FDE if .cfi_val_offset specifies non-default offset

Unwinding of the stack pointer (SP) is performed using the assumed
default rule ".cfi_val_offset <SP-reg>, 0", so that SP unwinds as:

  SP = CFA

Warn if the CFI directive .cfi_val_offset is encountered for the
SP register with a different offset.

gas/
* gen-sframe.c (sframe_xlate_do_val_offset): Skip SFrame FDE
if non-default SP value offset.

Signed-off-by: Jens Remus <jremus@linux.ibm.com>
4 months agogas: sframe: Use appropriate struct cfi_insn_data union members
Jens Remus [Tue, 4 Feb 2025 14:13:24 +0000 (15:13 +0100)] 
gas: sframe: Use appropriate struct cfi_insn_data union members

Use the appropriate struct cfi_insn_data union members to access
fields when generating SFrame information from CFI directives.

gas/
* gen-sframe.c (sframe_xlate_do_def_cfa, sframe_xlate_do_offset,
sframe_xlate_do_val_offset): Access ri fields, as .cfi_def_cfa,
.cfi_offset, and .cfi_val_offset define a register and offset
value.
* (sframe_xlate_do_def_cfa_register): Access r field, as
.cfi_def_cfa_register defines a register.

Signed-off-by: Jens Remus <jremus@linux.ibm.com>
4 months agogdb/python: add void_type () method to gdb.Architecture object
Jan Vrany [Tue, 4 Feb 2025 13:56:49 +0000 (13:56 +0000)] 
gdb/python: add void_type () method to gdb.Architecture object

This commit adds a new method to Python architecture objects that
returns a void type for that architecture.

This will be useful later to create types for function symbols created
using Python extension code.

Reviewed-By: Eli Zaretskii <eliz@gnu.org>
Approved-By: Andrew Burgess <aburgess@redhat.com>
4 months agogdb/python: add domain property to gdb.Symbol
Jan Vrany [Tue, 4 Feb 2025 13:56:49 +0000 (13:56 +0000)] 
gdb/python: add domain property to gdb.Symbol

Reviewed-By: Eli Zaretskii <eliz@gnu.org>
Approved-By: Andrew Burgess <aburgess@redhat.com>
4 months agogdb/python: add subblocks property to gdb.Block
Jan Vrany [Tue, 4 Feb 2025 13:56:49 +0000 (13:56 +0000)] 
gdb/python: add subblocks property to gdb.Block

This commit adds new propery "subblocks" to gdb.Block objects. This
allows Python to traverse block tree starting with global block.

Reviewed-By: Eli Zaretskii <eliz@gnu.org>
Approved-By: Andrew Burgess <aburgess@redhat.com>
4 months ago[gdb/testsuite] Fix gdb.ada/big_packed_array.exp on s390x-linux
Tom de Vries [Tue, 4 Feb 2025 13:06:20 +0000 (14:06 +0100)] 
[gdb/testsuite] Fix gdb.ada/big_packed_array.exp on s390x-linux

When running test-case gdb.ada/big_packed_array.exp on s390x-linux, I run
into:
...
(gdb) print bad^M
$2 = (0 => 0 <repeats 24 times>, 1)^M
(gdb) FAIL: gdb.ada/big_packed_array.exp: scenario=minimal: print bad
...

This is with gcc 7.5.0, and this xfail should trigger:
...
           if { $have_xfail && [string is integer $last] \
                   && [expr ($last & 0xf) == 0] } {
                # gcc/101643
                setup_xfail *-*-*
            }
...
but it doesn't because $last is '1'.

Fix this by using 0xf0 as mask for big endian.

Tested on s390x-linux.

4 months ago[gdb/testsuite] Fix gdb.ada/convvar_comp.exp on s390x-linux
Tom de Vries [Tue, 4 Feb 2025 13:06:20 +0000 (14:06 +0100)] 
[gdb/testsuite] Fix gdb.ada/convvar_comp.exp on s390x-linux

When running test-case gdb.ada/convvar_comp.exp on s390x-linux, I get:
...
(gdb) run ^M
Starting program: pb16_063 ^M
^M
Breakpoint 1, pck.break_me (item=...) at pck.adb:17^M
17        function Break_Me (Item : T) return Boolean is^M
(gdb) print item.started^M
Cannot access memory at address 0x0^M
(gdb) FAIL: gdb.ada/convvar_comp.exp: print item.started
...

This happens as follows.

The parameter item is available in (DW_OP_fbreg: -168):
...
 <2><912>: Abbrev Number: 18 (DW_TAG_formal_parameter)
    <913>   DW_AT_name        : (indirect string, offset: 0x14ca): item
    <919>   DW_AT_type        : <0x929>
    <91d>   DW_AT_location    : 3 byte block: 91 d8 7e  (DW_OP_fbreg: -168)
...
and according to the rules of -O0, it's considered to be available after the
prologue, which looks like this:
...
0000000001002998 <pck__break_me>:
 1002998:       b3 c1 00 2b             ldgr    %f2,%r11
 100299c:       b3 c1 00 0f             ldgr    %f0,%r15
 10029a0:       e3 f0 ff 58 ff 71       lay     %r15,-168(%r15)
 10029a6:       b9 04 00 bf             lgr     %r11,%r15
 10029aa:       e3 20 b0 a0 00 24       stg     %r2,160(%r11)
...

To detect the prologue, gdb checks the line info, which looks like this:
...
pck.adb:
File name                Line number    Starting address    View    Stmt
pck.adb                           17           0x1002998               x
pck.adb                           17           0x1002998       1       x
pck.adb                           19           0x10029b0               x
pck.adb                           20           0x10029b8               x
pck.adb                            -           0x10029c6
...
and gdb concludes that it's an empty prologue, so we stop at 0x1002998 and
try to print parameter item, which is not available yet.

For more details, see this comment in skip_prologue_using_sal:
...
  /* For languages other than assembly, treat two consecutive line
     entries at the same address as a zero-instruction prologue.
...

The same thing happens on x86_64-linux, but it causes no problem there,
because amd64_skip_prologue decides not to trust the result:
...
      struct compunit_symtab *cust = find_pc_compunit_symtab (func_addr);

      /* LLVM backend (Clang/Flang) always emits a line note before the
 prologue and another one after.  We trust clang and newer Intel
 compilers to emit usable line notes.  */
      if (post_prologue_pc
  && (cust != NULL
      && cust->producer () != nullptr
      && (producer_is_llvm (cust->producer ())
      || producer_is_icc_ge_19 (cust->producer ()))))
return std::max (start_pc, post_prologue_pc);
...
because the producer is GCC.

Work around this by setting a breakpoint on the first statement of
pck.break_me instead.

Tested on s390x-linux.

4 months ago[gdb/testsuite] Use c++ flag in c++ test-cases
Tom de Vries [Tue, 4 Feb 2025 12:53:20 +0000 (13:53 +0100)] 
[gdb/testsuite] Use c++ flag in c++ test-cases

In some cases, test-cases use c++, but don't add "c++" to the compilation
flags.  This can cause problems with some compilers.

Fix this in some test-cases.

Approved-By: Tom Tromey <tom@tromey.com>
PR testsuite/30380
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30380

4 months agoUpdate with latest changes to src-release.sh
Nick Clifton [Tue, 4 Feb 2025 11:50:32 +0000 (11:50 +0000)] 
Update with latest changes to src-release.sh

4 months agoRename 'binutils' to 'binutils_with_gold'. Rename 'bin_no_gold' to 'binutils'. ...
Nick Clifton [Tue, 4 Feb 2025 11:50:02 +0000 (11:50 +0000)] 
Rename 'binutils' to 'binutils_with_gold'.  Rename 'bin_no_gold' to 'binutils'.  Add 'gold'

4 months ago[gdb/testsuite] Fix gdb.base/list-dot-nodebug.exp on openSUSE
Tom de Vries [Tue, 4 Feb 2025 09:34:39 +0000 (10:34 +0100)] 
[gdb/testsuite] Fix gdb.base/list-dot-nodebug.exp on openSUSE

On openSUSE Leap 15.6 with test-case gdb.base/list-dot-nodebug.exp I run into:
...
(gdb) list .^M
warning: 1      ../sysdeps/x86_64/crtn.S: No such file or directory^M
(gdb) FAIL: $exp: debug=none: print before start
...

The intent of the debug=none case is to generate an executable with no debug
info.  However, we have quite a few CUs with debug info:
...
$ readelf -wi outputs/gdb.base/list-dot-nodebug/list-dot-nodebug-none \
    | egrep -c " @ "
431
...

This is because this code:
...
  gdb_gnu_strip_debug $executable no-debuglink
...
uses $executable, and the variable is set here:
...
set executable ${testfile}-none
...
which sets it to "list-dot-nodebug-none" and consequently
gdb_gnu_strip_debug cannot find it.

Fix this by using "[standard_output_file $executable]" instead.

Tested on x86_64-linux.

Approved-By: Tom Tromey <tom@tromey.com>
PR testsuite/31721
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=31721

4 months ago[gdb/tui] Remove stale title when showing "No Source Available"
Tom de Vries [Tue, 4 Feb 2025 09:21:28 +0000 (10:21 +0100)] 
[gdb/tui] Remove stale title when showing "No Source Available"

When running test-case gdb.tui/main.exp, the last command discards the
executable file and symbol table:
...
(gdb) file
No executable file now.
Discard symbol table from `main'? (y or n) [answered Y; input not from terminal]
No symbol file now.
(gdb)
...
and we end up with this source window:
...
+-tui-layout.c----------------------------------------------------------------+
|                                                                             |
|                                                                             |
|                                                                             |
|                                                                             |
|                                                                             |
|                                                                             |
|                           [ No Source Available ]                           |
|                                                                             |
|                                                                             |
|                                                                             |
|                                                                             |
|                                                                             |
|                                                                             |
+-----------------------------------------------------------------------------+
...

The source window title shouldn't be showing tui-layout.c.  It's the source
file containing function main for the executable that was just discarded.

Fix this by clearing the title in tui_source_window::erase_source_content.

Tested on x86_64-linux.

Approved-By: Tom Tromey <tom@tromey.com>
4 months agoelf: Store __ehdr_start hash in elf_link_hash_table
H.J. Lu [Mon, 3 Feb 2025 03:30:50 +0000 (11:30 +0800)] 
elf: Store __ehdr_start hash in elf_link_hash_table

Since

commit 97da0e2677c4a38df2406576428ec27d1da26e7c
Author: Alan Modra <amodra@gmail.com>
Date:   Wed Jan 12 23:42:23 2022 +1030

    tweak __ehdr_start visibility and flags for check_relocs

creates __ehdr_start hash in lang_symbol_tweaks, store __ehdr_start hash
in elf_link_hash_table so that we just need to lookup it up only once.

bfd/

* elf-bfd.h (elf_link_hash_table): Add hehdr_start.
* elf.c (assign_file_positions_for_load_sections): Use
hehdr_start.

ld/

* ldelf.c (ldelf_before_allocation): Use hehdr_start for
__ehdr_start hash.
* ldlang.c (lang_symbol_tweaks): Store hehdr_start hash in
hehdr_start.

Signed-off-by: H.J. Lu <hjl.tools@gmail.com>
4 months agoelflink.c: Replace bed->dynamic_sec_flags with flags
H.J. Lu [Mon, 3 Feb 2025 04:01:28 +0000 (12:01 +0800)] 
elflink.c: Replace bed->dynamic_sec_flags with flags

Since at the function entry, there is

  flags = bed->dynamic_sec_flags;

we can replace bed->dynamic_sec_flags with flags.

* elflink.c (_bfd_elf_create_got_section): Replace
bed->dynamic_sec_flags with flags.
(_bfd_elf_link_create_dynamic_sections): Likewise.

Signed-off-by: H.J. Lu <hjl.tools@gmail.com>
4 months agopre-commit autoupdate
Simon Marchi [Tue, 4 Feb 2025 01:55:04 +0000 (20:55 -0500)] 
pre-commit autoupdate

Run `pre-commit autoupdate`.

This picks up a fresh Black version from 2025, and with it comes a small
but welcome formatting change.

There is a new version of isort as well, but no formatting change there.

Change-Id: Ie654a9c14c3a4096893011082668efb57c166fa4

4 months agoAutomatic date update in version.in
GDB Administrator [Tue, 4 Feb 2025 00:00:14 +0000 (00:00 +0000)] 
Automatic date update in version.in

4 months ago[gdb/syscalls] Sync with strace v6.13
Tom de Vries [Mon, 3 Feb 2025 14:01:39 +0000 (15:01 +0100)] 
[gdb/syscalls] Sync with strace v6.13

After syncing with strace v6.13 using:
...
$ update-linux-defaults.sh ~/upstream/strace.git
...
we have a few new entries in linux-defaults.xml.in:
...
  <syscall name="getxattrat" groups="descriptor,file"/>
  <syscall name="listxattrat" groups="descriptor,file"/>
  <syscall name="removexattrat" groups="descriptor,file"/>
  <syscall name="setxattrat" groups="descriptor,file"/>
...

Regenerate most *-linux.xml.in files using:
...
$ ./update-linux-from-src.sh ~/upstream/linux-stable.git
...
updating the copyright years, and do so manually for the remaining two.

Then regenerate *-linux.xml using make, propagating the groups changes and
copyright years.

Tested on x86_64-linux.

Approved-By: Andrew Burgess <aburgess@redhat.com>
4 months agoZ8k: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:29:40 +0000 (12:29 +0100)] 
Z8k: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). At the same time use is_end_of_stmt() instead of an
open-coded check in adjacent code.

4 months agoZ80: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:29:29 +0000 (12:29 +0100)] 
Z80: use is_whitespace()

Replace an open-coded check and convert ISSPACE() uses.

4 months agoXtensa: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:29:10 +0000 (12:29 +0100)] 
Xtensa: use is_whitespace()

Convert an open-coded check.

4 months agoxgate: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:28:58 +0000 (12:28 +0100)] 
xgate: use is_whitespace()

Convert an open-coded check.

4 months agox86: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:28:46 +0000 (12:28 +0100)] 
x86: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agowasm32: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:28:22 +0000 (12:28 +0100)] 
wasm32: use is_whitespace()

Convert an open-coded check.

4 months agoVisium: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:26:00 +0000 (12:26 +0100)] 
Visium: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert an open-coded check.

4 months agoVAX: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:25:50 +0000 (12:25 +0100)] 
VAX: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agov850: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:25:41 +0000 (12:25 +0100)] 
v850: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks as well as ISSPACE()
uses. At the same time use is_end_of_stmt() instead of a kind-of-open-
coded check in adjacent code.

4 months agoC6x: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:25:28 +0000 (12:25 +0100)] 
C6x: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert an ISSPACE() use. At the same time use
is_end_of_stmt() instead of open-coded checks in adjacent code.

4 months agoC54x: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:25:10 +0000 (12:25 +0100)] 
C54x: use is_whitespace()

Convert ISSPACE() uses. At the same time use is_end_of_stmt() instead
of open-coded checks in adjacent code. The function also needs using in
next_line_shows_parallel().

4 months agoC4x: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:25:01 +0000 (12:25 +0100)] 
C4x: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). At the same time use is_end_of_stmt() instead of
kind-of-open-coded checks in adjacent code.

4 months agoC30: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:24:48 +0000 (12:24 +0100)] 
C30: use is_whitespace()

Convert an open-coded check.

4 months agospu: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:22:33 +0000 (12:22 +0100)] 
spu: use is_whitespace()

Convert ISSPACE() uses. At the same time use is_end_of_stmt() instead
of a kind-of-open-coded check in adjacent code.

4 months agoSparc: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:22:23 +0000 (12:22 +0100)] 
Sparc: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agoSH: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:22:10 +0000 (12:22 +0100)] 
SH: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks as well as an
ISSPACE() use. At the same time use is_end_of_stmt() instead of
(kind-of-)open-coded checks in adjacent code.

4 months agoScore: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:21:46 +0000 (12:21 +0100)] 
Score: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agoS/390: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:21:29 +0000 (12:21 +0100)] 
S/390: use is_whitespace()

Convert ISSPACE() uses. At the same time use is_end_of_stmt() instead
of kind-of-open-coded checks in adjacent code.

4 months agos12z: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:21:08 +0000 (12:21 +0100)] 
s12z: use is_whitespace()

Convert open-coded checks. At the same time use is_end_of_stmt() instead
of open-coded checks in adjacent code. This then also fixes the prior
use of a wrong cast for an array index: Plain char may, after all, be
signed.

4 months agorx: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:19:26 +0000 (12:19 +0100)] 
rx: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks as well as ISSPACE()
uses. At the same time use is_end_of_stmt() instead of an open-coded
check in adjacent code.

4 months agorl78: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:19:17 +0000 (12:19 +0100)] 
rl78: use is_whitespace()

Replace open-coded checks and convert ISSPACE() uses. At the same time
use is_end_of_stmt() instead of an open-coded check in adjacent code.

4 months agoRISC-V: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:19:05 +0000 (12:19 +0100)] 
RISC-V: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Switch places already checking for tabs to use the
macro, too.

4 months agopru: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:18:53 +0000 (12:18 +0100)] 
pru: use is_whitespace()

Convert open-coded checks as well as an ISSPACE() use.

4 months agoPPC: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:18:34 +0000 (12:18 +0100)] 
PPC: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also switch ISSPACE() uses over. At the same time
use is_end_of_stmt() instead of an open-coded nul char check.

4 months agoPicoJava: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:18:20 +0000 (12:18 +0100)] 
PicoJava: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert ISSPACE(). At the same time use
is_end_of_stmt() instead of an open-coded check in adjacent code.

4 months agoPDP11: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:18:07 +0000 (12:18 +0100)] 
PDP11: use is_whitespace()

Convert open-coded checks.

4 months agoNS32k: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:16:01 +0000 (12:16 +0100)] 
NS32k: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agonds32: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:15:48 +0000 (12:15 +0100)] 
nds32: use is_whitespace()

Convert ISSPACE() uses.

4 months agomsp430: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:15:35 +0000 (12:15 +0100)] 
msp430: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert ISSPACE() uses. At the same time use
is_end_of_stmt() instead of open-coded checking in code needing touching
anyway.

4 months agoMoxie: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:15:21 +0000 (12:15 +0100)] 
Moxie: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert ISSPACE() uses. At the same time use
is_end_of_stmt() instead of an open-coded check in adjacent code. While
at it also drop a redundant whitespace skipping loop.

4 months agomn10300: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:15:07 +0000 (12:15 +0100)] 
mn10300: use is_whitespace()

Convert open-coded checks as well as ISSPACE() uses. At the same time
use is_end_of_stmt() instead of kind-of-open-coded checks in adjacent
code.

4 months agomn10200: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:14:56 +0000 (12:14 +0100)] 
mn10200: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks as well as ISSPACE()
uses. At the same time use is_end_of_stmt() instead of kind-of-open-
coded checks in adjacent code.

4 months agoMIPS: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:07:21 +0000 (12:07 +0100)] 
MIPS: use is_whitespace()

... for consistency of recognition of what is deemed whitespace.

At the same time use is_end_of_stmt() instead of an open-coded nul char
check, and check for statement end in the first place in
parse_relocation().

4 months agoMicroBlaze: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:07:09 +0000 (12:07 +0100)] 
MicroBlaze: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert ISSPACE() uses. At the same time use
is_end_of_stmt() instead of an open-coded check in adjacent code.

4 months agometag: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:06:57 +0000 (12:06 +0100)] 
metag: use is_whitespace()

Replace the custom is_whitespace_char().

4 months agoM*Core: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:06:43 +0000 (12:06 +0100)] 
M*Core: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert ISSPACE() uses. At the same time use
is_end_of_stmt() instead of an open-coded check in adjacent code.

4 months agoM68k: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:06:34 +0000 (12:06 +0100)] 
M68k: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks where tabs were
already included. At the same time use is_end_of_stmt() instead of open-
coded checks in adjacent code.

4 months agoM68HC1x: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:06:21 +0000 (12:06 +0100)] 
M68HC1x: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks where tabs were
already included. At the same time use is_end_of_stmt() instead of an
open-coded check in adjacent code.

4 months agom32r: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:06:06 +0000 (12:06 +0100)] 
m32r: use is_whitespace()

Convert a lonely ISSPACE().

4 months agom32c: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:05:55 +0000 (12:05 +0100)] 
m32c: use is_whitespace()

Convert open-coded checks as well as the sole ISBLANK() use throughout
the gas/ tree.

4 months agoLoongArch: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:05:33 +0000 (12:05 +0100)] 
LoongArch: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agokvx: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:03:07 +0000 (12:03 +0100)] 
kvx: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks where tabs were
already included. At the same time use is_end_of_stmt() instead of open-
coded checks in adjacent code.

4 months agoHP-PA: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:02:50 +0000 (12:02 +0100)] 
HP-PA: use is_whitespace()

Convert open-coded checks. At the same time use is_end_of_stmt() instead
of an open-coded check in adjacent code.

4 months agoH8/300: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:02:38 +0000 (12:02 +0100)] 
H8/300: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). At the same time use is_end_of_stmt() instead of an
open-coded check in adjacent code.

4 months agoft32: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:02:09 +0000 (12:02 +0100)] 
ft32: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also switch ISSPACE() uses over. At the same time
use is_end_of_stmt() instead of open-coded checks in adjacent code.

4 months agofr30: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 11:01:59 +0000 (12:01 +0100)] 
fr30: use is_whitespace()

Convert open-coded checks. At the same time use is_end_of_stmt() instead
of an open-coded check in adjacent code.

4 months agoEpiphany: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:59:38 +0000 (11:59 +0100)] 
Epiphany: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agoCRx: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:59:22 +0000 (11:59 +0100)] 
CRx: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also switch ISSPACE() uses over.

4 months agocris: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:59:00 +0000 (11:59 +0100)] 
cris: use is_whitespace()

Switch ISSPACE() uses over.

Unlike many other targets, limiting whitespace checks to just blanks is
deemed okay here: Compilers wanting to use -f / #NO_APP are apparently
required to emit only blanks (without this being written down anywhere).

4 months agoCR16: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:58:49 +0000 (11:58 +0100)] 
CR16: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also switch ISSPACE() uses over.

4 months agoC-Sky: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:58:32 +0000 (11:58 +0100)] 
C-Sky: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also switch ISSPACE() uses over. At the same time
use is_end_of_stmt() instead of kind-of-open-coded checks.

4 months agodlx: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:56:25 +0000 (11:56 +0100)] 
dlx: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks where tabs were
already included.

4 months agod30v: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:56:13 +0000 (11:56 +0100)] 
d30v: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks where tabs were
already included. At the same time use is_end_of_stmt() instead of open-
coded checks in adjacent code.

4 months agod10v: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:56:04 +0000 (11:56 +0100)] 
d10v: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Also convert open-coded checks where tabs were
already included. At the same time use is_end_of_stmt() instead of open-
coded checks in adjacent code.

4 months agobpf: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:55:54 +0000 (11:55 +0100)] 
bpf: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). Various redundant nul char checks are also dropped,
where adjacent. At the same time use is_end_of_stmt() instead of an
open-coded nul char check.

4 months agobfin: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:55:40 +0000 (11:55 +0100)] 
bfin: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agogas/obj-*.c: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:55:23 +0000 (11:55 +0100)] 
gas/obj-*.c: use is_whitespace()

... for consistency of recognition of what is deemed whitespace.

In obj_elf_section_name() also generalize end-of-statement recognition
at the same time. Conversely drop the unused SKIP_SEMI_COLON() for COFF.

4 months agoavr: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:50:43 +0000 (11:50 +0100)] 
avr: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agoaarch64: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:50:30 +0000 (11:50 +0100)] 
aarch64: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input).

4 months agoArm: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:50:20 +0000 (11:50 +0100)] 
Arm: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). At the same time use is_end_of_stmt() instead of an
open-coded nul char check.

In parse_neon_type() be more aggressive and remove the special casing of
certain characters altogether. The original default case simply having
"break" can't have been correct.

4 months agoarc: use is_whitespace()
Jan Beulich [Mon, 3 Feb 2025 10:50:03 +0000 (11:50 +0100)] 
arc: use is_whitespace()

Wherever blanks are permissible in input, tabs ought to be permissible,
too. This is particularly relevant when -f is passed to gas (alongside
appropriate input). At the same time use is_end_of_stmt() instead of
open-coded nul char checks.

4 months agoAlpha/EVAX: use is_whitespace() / is_end_of_stmt()
Jan Beulich [Mon, 3 Feb 2025 10:49:48 +0000 (11:49 +0100)] 
Alpha/EVAX: use is_whitespace() / is_end_of_stmt()

Don't open-code checking for ' ', '\t', and statement ending chars.

4 months agogas: consolidate whitespace recognition
Jan Beulich [Mon, 3 Feb 2025 10:48:55 +0000 (11:48 +0100)] 
gas: consolidate whitespace recognition

Let's extend lex_type[] to also cover whitespace, then having a simple
macro to uniformly recognize both blanks and tabs (and \r when it's not
EOL) as such.

In macro.c use sb_skip_white() as appropriate, instead of open-coding
it.

4 months agoAutomatic date update in version.in
GDB Administrator [Mon, 3 Feb 2025 00:00:08 +0000 (00:00 +0000)] 
Automatic date update in version.in

4 months agoAvoid "text file busy" in dw2-using-debug-str.exp
Tom Tromey [Sat, 1 Feb 2025 23:51:42 +0000 (16:51 -0700)] 
Avoid "text file busy" in dw2-using-debug-str.exp

When I run:

    runtest dw2-using-debug-str.exp

... if I examine the gdb.log, I see:

    objcopy: unable to copy file '[...]/dw2-using-debug-str'; reason: Text file busy

This happens because the inferior is still running, and objcopy --
despite the invocation seemingly not needing this -- tries to open it
for writing.

This patch works around the objcopy oddity by having gdb exit (killing
the inferior) before the invocation.

Fixing this points out that the test does not work in the
--target_board=cc-with-gdb-index case.  This patch also arranges to
issue an "untested" here.

4 months agoAutomatic date update in version.in
GDB Administrator [Sun, 2 Feb 2025 00:00:10 +0000 (00:00 +0000)] 
Automatic date update in version.in

4 months agoRemove obsolete test from gdb.cp/var-tag.exp
Tom Tromey [Sat, 4 Jan 2025 17:48:07 +0000 (10:48 -0700)] 
Remove obsolete test from gdb.cp/var-tag.exp

There is a test in gdb.cp/var-tag.exp that is kfail'd.  I happened
across this while working on another series and found that the PR it
referenced was closed as invalid.  On that basis I think the test
should be deleted.

Reviewed-By: Keith Seitz <keiths@redhat.com>
4 months agoShow type- and function-domain in maint print psymbols
Tom Tromey [Mon, 20 Jan 2025 18:02:51 +0000 (11:02 -0700)] 
Show type- and function-domain in maint print psymbols

I neglected to update "maint print psymbols" when adding TYPE_DOMAIN
and FUNCTION_DOMAIN.  This would have been mildly helpful when
debugging a series I am working on.  This patch corrects the
oversight.

Approved-By: Andrew Burgess <aburgess@redhat.com>
4 months agoAutomatic date update in version.in
GDB Administrator [Sat, 1 Feb 2025 00:00:14 +0000 (00:00 +0000)] 
Automatic date update in version.in

4 months agoUse "false" when setting cli_styling
Tom Tromey [Fri, 31 Jan 2025 21:23:32 +0000 (14:23 -0700)] 
Use "false" when setting cli_styling

I noticed a spot that uses 0 where "false" is more appropriate.

4 months agoAdd space in name of Rust tuple type
Tom Tromey [Sun, 19 Jan 2025 23:57:07 +0000 (16:57 -0700)] 
Add space in name of Rust tuple type

The Rust compiler emits tuple type names with a space after the comma,
like "(i32, f64)".  This changes rust-parse.c to follow.  This isn't
ideal -- probably the DWARF reader should canonicalize these names --
but it is a bit more robust if symbol lookup should change; and anyway
this feature of gdb is probably rarely used.

4 months agoaarch64: Support +sme+nosve permissively
Andrew Carlotti [Fri, 31 Jan 2025 05:07:30 +0000 (05:07 +0000)] 
aarch64: Support +sme+nosve permissively

There is inconsistency regarding whether or not +sme implies +sve2 and
whether +nosve2 implies +nosme.  In particular, GCC 14 assumes the
dependency exists, and canonicalises target strings accordingly, whereas
LLVM treats the features as independent.

This patch removes the positive implication while retaining the negative
implication.  This is the more permissive choice in each case, and
allows us to support target strings written with either interpretation
in mind.

This reduces our ability to detect invalid instructions, but we already
can't rely on this detection because gas doesn't know whether functions
might be executed in streaming mode and/or non-streaming mode.

The aarch64_feature_enable_set change is functionally redundant within
this patch.  It is included because the longer term intention is to
instead remove the workaround in aarch64_parse_features, once the
internal feature checks have been modified to support having both
AARCH64_FEATURE_SME set and AARCH64_FEATURE_SVE unset.

Similarly, the dependency from +sme to +fp16 is currently redundant, but
this redundancy relies upon an incorrect dependency from +fcma to +fp16.
This can be fixed in the future, but it might require modifying internal
feature checks for a few FCMA instructions, so it's left unchanged for
now.

4 months agoaarch64: Fix fp8 feature dependencies
Andrew Carlotti [Thu, 30 Jan 2025 19:14:46 +0000 (19:14 +0000)] 
aarch64: Fix fp8 feature dependencies

We agreed with LLVM that we shouldn't enforce the architectural
dependencies between fp8 muliplication features, so remove them.

Additionally, fix a typo in the gating for FEAT_SME_F8F16 instructions,
which were mistakenly gated by +sme-f8f32 instead.  Until now this
mistake had been masked by the dependency between the features.

4 months agoaarch64: Fix overly lax +frintts dependency
Andrew Carlotti [Thu, 30 Jan 2025 18:35:14 +0000 (18:35 +0000)] 
aarch64: Fix overly lax +frintts dependency

We agreed with LLVM that +frintts should only enable +fp, not +simd.
This also matches the dependency used in GCC.

4 months agoLoongArch: Do not relax against __[start|stop]_SECNAME symbol
Lulu Cai [Fri, 31 Jan 2025 10:37:50 +0000 (10:37 +0000)] 
LoongArch: Do not relax against __[start|stop]_SECNAME symbol

4 months agox86/APX: correct libbfd's EVEX Rn -> Bn transformations
Jan Beulich [Fri, 31 Jan 2025 09:07:54 +0000 (10:07 +0100)] 
x86/APX: correct libbfd's EVEX Rn -> Bn transformations

In the recent GOTPCREL addition I screwed up, in clearing the Rn bits
afterwards rather than setting them. While that ought to be benign (for
the bits being ignored in situations like this), we still want to leave
"canonical" encodings.

The pre-existing GOTTPOFF conversion wasn't doing quite correctly
either: We cannot assume the incoming Bn bits to be in a particular
state, as for the addressing form in question they're ignored as well.

To address both, introduce a helper function. This is then also an
overall reduction of (source) code size (and use of "magic" numbers).

4 months agox86/APX: GETSEC cannot be used with REX2
Jan Beulich [Fri, 31 Jan 2025 09:07:32 +0000 (10:07 +0100)] 
x86/APX: GETSEC cannot be used with REX2

It lives in a "forbidden" row, yet its disassembler table entry was
lacking a respective marker.

4 months agox86: support RMPREAD insn
Jan Beulich [Fri, 31 Jan 2025 09:06:02 +0000 (10:06 +0100)] 
x86: support RMPREAD insn

Like for RMPUPDATE documentation is about to change as far as operands
are concerned. They're merely the other way around here.

While adjustind gas documentation, also add the missing RMPQUERY
counterparts there.

4 months agox86: RMPUPDATE wants operands in different form
Jan Beulich [Fri, 31 Jan 2025 09:05:36 +0000 (10:05 +0100)] 
x86: RMPUPDATE wants operands in different form

AMD are about to update their doc, to help clarify that what we
currently do isn't quite right: In particular it is not %rax but %rcx
which is affected by address size. In fact, that's a normal memory
operand, just not expressed via ModR/M byte, but fixed to (%rcx) (or
(%ecx) with 32-bit addressing).

To support this in the assembler, generalize memory operand handling so
far specific to XLAT (which isn't really a string insn, but requires its
memory operand to be (%bx) / (%ebx) / (%rbx)).

In the disassembler mimic handling after XLAT's, too.

4 months agox86-64: omit "default" segment prefixes from string insn disassembly
Jan Beulich [Fri, 31 Jan 2025 09:04:45 +0000 (10:04 +0100)] 
x86-64: omit "default" segment prefixes from string insn disassembly

Printing implicit %ds: and %es: prefixes is pretty meaningless in 64-bit
mode. The SDM explicitly omits them for the 64-bit forms, and it
obviously has them for the other ones only to cover non-64-bit modes
(oddly enough the AMD PM has them present).

4 months agoRISC-V: widen LEB128 support
Jan Beulich [Fri, 31 Jan 2025 09:04:01 +0000 (10:04 +0100)] 
RISC-V: widen LEB128 support

Do away with at least one of the limitations - all other targets permit
multiple values to be specified with a single directive. Re-arrange the
logic further to also overcome an internal error in
riscv_insert_uleb128_fixes(), as e.g. observed by the all/sleb128-2
testcase. This way there's also no need to parse expressions twice,
thus also not raising the same diagnostics (if any) twice.

Note how this addresses a pre-existing XFAIL (where the comment wasn't
really applicable either for RISC-V).

Also update documentation, also to mention that differences between
symbols may be used with .uleb128 (albeit I'm uncertain whether there
are limitations).