binutils-gdb.git
2 years agoConvert wrap_here to use integer parameter
Tom Tromey [Thu, 30 Dec 2021 16:46:02 +0000 (09:46 -0700)]
Convert wrap_here to use integer parameter

I think it only really makes sense to call wrap_here with an argument
consisting solely of spaces.  Given this, it seemed better to me that
the argument be an int, rather than a string.  This patch is the
result.  Much of it was written by a script.

2 years agogdb/python: improve the auto help text for gdb.Parameter
Andrew Burgess [Fri, 7 Jan 2022 16:48:49 +0000 (16:48 +0000)]
gdb/python: improve the auto help text for gdb.Parameter

This commit attempts to improve the help text that is generated for
gdb.Parameter objects when the user fails to provide their own
documentation.

Documentation for a gdb.Parameter is currently pulled from two
sources: the class documentation string, and the set_doc/show_doc
class attributes.  Thus, a fully documented parameter might look like
this:

  class Param_All (gdb.Parameter):
     """This is the class documentation string."""

     show_doc = "Show the state of this parameter"
     set_doc = "Set the state of this parameter"

     def get_set_string (self):
        val = "on"
        if (self.value == False):
           val = "off"
        return "Test Parameter has been set to " + val

     def __init__ (self, name):
        super (Param_All, self).__init__ (name, gdb.COMMAND_DATA, gdb.PARAM_BOOLEAN)
        self._value = True

  Param_All ('param-all')

Then in GDB we see this:

  (gdb) help set param-all
  Set the state of this parameter
  This is the class documentation string.

Which is fine.  But, if the user skips both of the documentation parts
like this:

  class Param_None (gdb.Parameter):

     def get_set_string (self):
        val = "on"
        if (self.value == False):
           val = "off"
        return "Test Parameter has been set to " + val

     def __init__ (self, name):
        super (Param_None, self).__init__ (name, gdb.COMMAND_DATA, gdb.PARAM_BOOLEAN)
        self._value = True

  Param_None ('param-none')

Now in GDB we see this:

  (gdb) help set param-none
  This command is not documented.
  This command is not documented.

That's not great, the duplicated text looks a bit weird.  If we drop
different parts we get different results.  Here's what we get if the
user drops the set_doc and show_doc attributes:

  (gdb) help set param-doc
  This command is not documented.
  This is the class documentation string.

That kind of sucks, we say it's undocumented, then proceed to print
the documentation.  Finally, if we drop the class documentation but
keep the set_doc and show_doc:

  (gdb) help set param-set-show
  Set the state of this parameter
  This command is not documented.

That seems OK.

So, I think there's room for improvement.

With this patch, for the four cases above we now see this:

  # All values provided by the user, no change in this case:
  (gdb) help set param-all
  Set the state of this parameter
  This is the class documentation string.

  # Nothing provided by the user, the first string is now different:
  (gdb) help set param-none
  Set the current value of 'param-none'.
  This command is not documented.

  # Only the class documentation is provided, the first string is
  # changed as in the previous case:
  (gdb) help set param-doc
  Set the current value of 'param-doc'.
  This is the class documentation string.

  # Only the set_doc and show_doc are provided, this case is unchanged
  # from before the patch:
  (gdb) help set param-set-show
  Set the state of this parameter
  This command is not documented.

The one place where this change might be considered a negative is when
dealing with prefix commands.  If we create a prefix command but don't
supply the set_doc / show_doc strings, then this is what we saw before
my patch:

  (gdb) python Param_None ('print param-none')
  (gdb) help set print
  set print, set pr, set p
  Generic command for setting how things print.

  List of set print subcommands:

  ... snip ...
  set print param-none -- This command is not documented.
  ... snip ...

And after my patch:

  (gdb) python Param_None ('print param-none')
  (gdb) help set print
  set print, set pr, set p
  Generic command for setting how things print.

  List of set print subcommands:

  ... snip ...
  set print param-none -- Set the current value of 'print param-none'.
  ... snip ...

This seems slightly less helpful than before, but I don't think its
terrible.

Additionally, I've changed what we print when the get_show_string
method is not provided in Python.

Back when gdb.Parameter was first added to GDB, we didn't provide a
show function when registering the internal command object within
GDB.  As a result, GDB would make use of its "magic" mangling of the
show_doc string to create a sentence that would display the current
value (see deprecated_show_value_hack in cli/cli-setshow.c).

However, when we added support for the get_show_string method to
gdb.Parameter, there was an attempt to maintain backward compatibility
by displaying the show_doc string with the current value appended, see
get_show_value in py-param.c.  Unfortunately, this isn't anywhere
close to what deprecated_show_value_hack does, and the results are
pretty poor, for example, this is GDB before my patch:

  (gdb) show param-none
  This command is not documented. off

I think we can all agree that this is pretty bad.

After my patch, we how show this:

  (gdb) show param-none
  The current value of 'param-none' is "off".

Which at least is a real sentence, even if it's not very informative.

This patch does change the way that the Python API behaves slightly,
but only in the cases when the user has missed providing GDB with some
information.  In most cases I think the new behaviour is a lot better,
there's the one case (noted above) which is a bit iffy, but I think is
still OK.

I've updated the existing gdb.python/py-parameter.exp test to cover
the modified behaviour.

Finally, I've updated the documentation to (I hope) make it clearer
how the various bits of help text come together.

2 years agogdb/python: add gdb.history_count function
Andrew Burgess [Mon, 24 Jan 2022 15:19:43 +0000 (15:19 +0000)]
gdb/python: add gdb.history_count function

Add a new function gdb.history_count to the Python api, this function
returns an integer, the number of items in GDB's value history.

This is useful if you want to pull items from the history by their
absolute number, for example, if you wanted to show a complete history
list.  Previously we could figure out how many items are in the
history list by trying to fetch the items, and then catching the
exception when the item is not available, but having this function
seems nicer.

2 years agoRemove unused declaration
Tom Tromey [Wed, 26 Jan 2022 21:26:44 +0000 (14:26 -0700)]
Remove unused declaration

This removes an unused declaration from top.h.  This type is not
defined anywhere.

2 years agogdb: convert maintenance target-async and target-non-stop settings to callbacks
Simon Marchi [Thu, 25 Nov 2021 03:11:58 +0000 (22:11 -0500)]
gdb: convert maintenance target-async and target-non-stop settings to callbacks

This simplifies things a bit, as we don't need two variables and think
about reverting target_async_permitted_1 and target_non_stop_enabled_1
values if we can't change the setting.

Change-Id: I36acab045dacf02ae1988486cfdb27c1dff309f6

2 years agoReference array of structs instead of first member during memcpy
Keith Seitz [Wed, 26 Jan 2022 16:56:18 +0000 (08:56 -0800)]
Reference array of structs instead of first member during memcpy

aarch64-tdep.c defines the following macro:

#define MEM_ALLOC(MEMS, LENGTH, RECORD_BUF) \
        do  \
          { \
            unsigned int mem_len = LENGTH; \
            if (mem_len) \
              { \
                MEMS =  XNEWVEC (struct aarch64_mem_r, mem_len);  \
                memcpy(&MEMS->len, &RECORD_BUF[0], \
                       sizeof(struct aarch64_mem_r) * LENGTH); \
              } \
          } \
          while (0)

This is simlpy allocating a new array and copying it. However, for
the destination address, it is actually copying into the first member
of the first element of the array (`&MEMS->len"). This elicits a
warning with GCC 12:

../../binutils-gdb/gdb/aarch64-tdep.c: In function ‘int aarch64_process_record(gdbarch*, regcache*, CORE_ADDR)’:
../../binutils-gdb/gdb/aarch64-tdep.c:3711:23: error: writing 16 bytes into a region of size 8 [-Werror=stringop-overflow=]
 3711 |                 memcpy(&MEMS->len, &RECORD_BUF[0], \
      |                       ^
../../binutils-gdb/gdb/aarch64-tdep.c:4394:3: note: in expansion of macro ‘MEM_ALLOC’
 4394 |   MEM_ALLOC (aarch64_insn_r->aarch64_mems, aarch64_insn_r->mem_rec_count,
      |   ^~~~~~~~~
../../binutils-gdb/gdb/aarch64-tdep.c:3721:12: note: destination object ‘aarch64_mem_r::len’ of size 8
 3721 |   uint64_t len;    /* Record length.  */
      |            ^~~

The simple fix is to reference the array, `MEMS' as the destination of the copy.

Tested by rebuilding.

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Tue Jan 25 08:28:32 2022 -0800
#
# On branch master
# Your branch is ahead of 'origin/master' by 1 commit.
#   (use "git push" to publish your local commits)
#
# Changes to be committed:
# modified:   aarch64-tdep.c
#

2 years agogdb: add string_file::release method
Simon Marchi [Tue, 25 Jan 2022 01:00:46 +0000 (20:00 -0500)]
gdb: add string_file::release method

A common pattern for string_file is to want to move out the internal
string buffer, because it is the result of the computation that we want
to return.  It is the reason why string_file::string returns a non-const
reference, as explained in the comment.  I think it would make sense to
have a dedicated method for that instead and make string_file::string
return a const reference.

This allows removing the explicit std::move in the typical case.  Note
that compile_program::compute was missing a move, meaning that the
resulting string was copied.  With the new version, it's not possible to
forget to move.

Change-Id: Ieaefa35b73daa7930b2f3a26988b6e3b4121bb79

2 years agoAdd a way to temporarily set a gdb parameter from Python
Tom Tromey [Tue, 4 Jan 2022 18:00:52 +0000 (11:00 -0700)]
Add a way to temporarily set a gdb parameter from Python

It's sometimes useful to temporarily set some gdb parameter from
Python.  Now that the 'endian' crash is fixed, and now that the
current language is no longer captured by the Python layer, it seems
reasonable to add a helper function for this situation.

This adds a new gdb.with_parameter function.  This creates a context
manager which temporarily sets some parameter to a specified value.
The old value is restored when the context is exited.  This is most
useful with the Python "with" statement:

   with gdb.with_parameter('language', 'ada'):
      ... do Ada stuff

This also adds a simple function to set a parameter,
gdb.set_parameter, as suggested by Andrew.

This is PR python/10790.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=10790

2 years agoFix another crash with gdb parameters in Python
Tom Tromey [Tue, 4 Jan 2022 15:52:40 +0000 (08:52 -0700)]
Fix another crash with gdb parameters in Python

While looking into the language-capturing issue, I found another way
to crash gdb using parameters from Python:

(gdb) python print(gdb.parameter('endian'))

(This is related to PR python/12188, though this patch isn't going to
fix what that bug is really about.)

The problem here is that the global variable that underlies the
"endian" parameter is initialized to NULL.  However, that's not a
valid value for an "enum" set/show parameter.

My understanding is that, in gdb, an "enum" parameter's underlying
variable must have a value that is "==" (not just strcmp-equal) to one
of the values coming from the enum array.  This invariant is relied on
in various places.

I started this patch by fixing the problem with "endian".  Then I
added some assertions to add_setshow_enum_cmd to try to catch other
problems of the same type.

This patch fixes all the problems that I found.  I also looked at all
the calls to add_setshow_enum_cmd to ensure that they were all
included in the gdb I tested.  I think they are: there are no calls in
nat-* files, or in remote-sim.c; and I was trying a build with all
targets, Python, and Guile enabled.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=12188

2 years agoChange how Python architecture and language are handled
Tom Tromey [Tue, 4 Jan 2022 15:02:24 +0000 (08:02 -0700)]
Change how Python architecture and language are handled

Currently, gdb's Python layer captures the current architecture and
language when "entering" Python code.  This has some undesirable
effects, and so this series changes how this is handled.

First, there is code like this:

  gdbpy_enter enter_py (python_gdbarch, python_language);

This is incorrect, because both of these are NULL when not otherwise
assigned.  This can cause crashes in some cases -- I've added one to
the test suite.  (Note that this crasher is just an example, other
ones along the same lines are possible.)

Second, when the language is captured in this way, it means that
Python code cannot affect the current language for its own purposes.
It's reasonable to want to write code like this:

    gdb.execute('set language mumble')
    ... stuff using the current language
    gdb.execute('set language previous-value')

However, this won't actually work, because the language is captured on
entry.  I've added a test to show this as well.

This patch changes gdb to try to avoid capturing the current values.
The Python concept of the current gdbarch is only set in those few
cases where a non-default value is computed or needed; and the
language is not captured at all -- instead, in the cases where it's
required, the current language is temporarily changed.

2 years agobfd: Make bfd.stamp depend on source bfd.texi
H.J. Lu [Tue, 25 Jan 2022 17:08:43 +0000 (09:08 -0800)]
bfd: Make bfd.stamp depend on source bfd.texi

Make bfd.stamp depend on source bfd.texi to avoid regenerating
doc/bfd.info for each make run.

PR binutils/28807
* Makefile.in: Regenerate.
* doc/local.mk (%D%/bfd.stamp): Depend on $(srcdir)/%D%/bfd.texi.

2 years agold: Rewrite lang_size_relro_segment_1
H.J. Lu [Fri, 14 Jan 2022 21:48:36 +0000 (13:48 -0800)]
ld: Rewrite lang_size_relro_segment_1

1. Compute the desired PT_GNU_RELRO segment base and find the maximum
section alignment of sections starting from the PT_GNU_RELRO segment.
2. Find the first preceding load section.
3. Don't add the 1-page gap between the first preceding load section and
the relro segment if the maximum page size >= the maximum section
alignment.  Align the PT_GNU_RELRO segment first.  Subtract the maximum
page size if therer is still a 1-page gap.

PR ld/28743
PR ld/28819
* ldlang.c (lang_size_relro_segment_1): Rewrite.
* testsuite/ld-x86-64/pr28743-1.d: New file.
* testsuite/ld-x86-64/pr28743-1.s: Likewise.
* testsuite/ld-x86-64/x86-64.exp: Run pr28743-1.

2 years agogdb/testsuite: Ensure constant test name in gdb.base/break-interp.exp
Lancelot SIX [Wed, 19 Jan 2022 10:44:06 +0000 (05:44 -0500)]
gdb/testsuite: Ensure constant test name in gdb.base/break-interp.exp

When running the testsuite, I have lines similar to the following in the
gdb.sum file:

~~~
PASS: gdb.base/break-interp.exp: ldprelink=NO: ldsepdebug=NO: first backtrace: p /x 0x7f283d2f0fd1
...
PASS: gdb.base/break-interp.exp: ldprelink=NO: ldsepdebug=NO: binprelink=NO: binsepdebug=NO: binpie=NO: INNER: first backtrace: p /x 0x7f00de0317a5
...
~~~

The address part of the command might change between execution of the
test, which adds noise to a diff between two .sum files.

This patch changes to test name to "p /x $pc" in order to have constant
test name.

Tested on x86_64-Linux.

Change-Id: I973c1237a084dd6d424276443cbf0920533c9a21

2 years agoAutomatic date update in version.in
GDB Administrator [Wed, 26 Jan 2022 00:00:19 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agoAlways print the "host libthread-db" message to stdout
Tom Tromey [Thu, 30 Dec 2021 19:13:52 +0000 (12:13 -0700)]
Always print the "host libthread-db" message to stdout

linux-thread-db.c has a bit of unusual code that unconditionally
prints a message, but decides whether to print to gdb_stdout or
gdb_stdlog based on a debug flag.  It seems better to me to simply
always print this; and this is the only spot in gdb where we
conditionally pass gdb_stdout to one of the f*_unfiltered functions.

2 years agoReduce explicit use of gdb_stdout
Tom Tromey [Fri, 31 Dec 2021 00:31:41 +0000 (17:31 -0700)]
Reduce explicit use of gdb_stdout

In an earlier version of the pager rewrite series, it was important to
audit unfiltered output calls to see which were truly necessary.

This is no longer necessary, but it still seems like a decent cleanup
to change calls to avoid explicitly passing gdb_stdout.  That is,
rather than using something like fprintf_unfiltered with gdb_stdout,
the code ought to use plain printf_unfiltered instead.

This patch makes this change.  I went ahead and converted all the
_filtered calls I could find, as well, for the same clarity.

2 years agoSent timing stats to gdb_stdlog
Tom Tromey [Wed, 29 Dec 2021 21:16:02 +0000 (14:16 -0700)]
Sent timing stats to gdb_stdlog

This changes the time / space / symtab per-command statistics code to
send its output to gdb_stdlog rather than gdb_stdout.  This seems
slightly more correct to me.

2 years agoSend some error output to gdb_stderr
Tom Tromey [Wed, 29 Dec 2021 19:50:25 +0000 (12:50 -0700)]
Send some error output to gdb_stderr

This changes some code to send some error messages to gdb_stderr
rather than gdb_stdout.

2 years agoFix a probem building the binutils on SPARC/amd64
Klaus Ziegler [Tue, 25 Jan 2022 17:33:03 +0000 (17:33 +0000)]
Fix a probem building the binutils on SPARC/amd64

PR 28816
* elf/common.h (AT_SUN_HWCAP): Make definition conditional.

2 years agobfd: Regenerate Makefile.in
H.J. Lu [Tue, 25 Jan 2022 16:54:36 +0000 (08:54 -0800)]
bfd: Regenerate Makefile.in

* Makefile.in: Regenerate.

2 years agogold: drop old cygnus install hack
Mike Frysinger [Sun, 23 Jan 2022 17:53:22 +0000 (12:53 -0500)]
gold: drop old cygnus install hack

The gold subdir doesn't actually have a manual, so this hack doesn't
do anything.  Plus the automake cygnus option was removed years ago
by Simon in d0ac1c44885daf68f631befa37e ("Bump to autoconf 2.69 and
automake 1.15.1").  So delete it here.

2 years agogas: drop old cygnus install hack
Mike Frysinger [Sun, 23 Jan 2022 17:44:24 +0000 (12:44 -0500)]
gas: drop old cygnus install hack

This was needed when gas was using the automake cygnus option, but
this was removed years ago by Simon in d0ac1c44885daf68f631befa37e
("Bump to autoconf 2.69 and automake 1.15.1").  So delete it here.
The info pages are already & still installed by default w/out it.

2 years agoAutomatic date update in version.in
GDB Administrator [Tue, 25 Jan 2022 00:00:17 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agobfd: Update doc/local.mk
H.J. Lu [Mon, 24 Jan 2022 15:39:44 +0000 (07:39 -0800)]
bfd: Update doc/local.mk

PR binutils/28807
* Makefile.in: Regenerate.
* doc/local.mk (AM_MAKEINFOFLAGS): Add -I "$(srcdir)/%D%" -I %D%.
(TEXI2DVI): New.
(%D%/bfd.texi): Removed.
(doc/bfd/index.html): Remove -I$(srcdir).  Replace bfd.texi with
%D%/bfd.texi.

2 years agobfd/doc: Fix racy build failure from missing mkdir
Roland McGrath [Mon, 24 Jan 2022 20:38:50 +0000 (12:38 -0800)]
bfd/doc: Fix racy build failure from missing mkdir

bfd/
* doc/local.mk (%D%/bfdver.texi): Add mkdir command.

2 years agoFix a proble building the libiberty library with gcc-12.
Martin Sebor [Mon, 24 Jan 2022 17:56:23 +0000 (17:56 +0000)]
Fix a proble building the libiberty library with gcc-12.

PR 28779
* regex.c: Suppress -Wuse-after-free.

2 years agogdb/doc: improve description for Window.click on Python TUI windows
Andrew Burgess [Mon, 24 Jan 2022 10:33:23 +0000 (10:33 +0000)]
gdb/doc: improve description for Window.click on Python TUI windows

The description of the Window.click method doesn't mention where the
coordinates are anchored (it's the top left corner).

This minor tweak just mentions this point.

2 years agoUpdate Bulgarian, French, Romaniam and Ukranian translation for some of the sub-direc...
Nick Clifton [Mon, 24 Jan 2022 14:22:49 +0000 (14:22 +0000)]
Update Bulgarian, French, Romaniam and Ukranian translation for some of the sub-directories

2 years agoAutomatic date update in version.in
GDB Administrator [Mon, 24 Jan 2022 00:00:15 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agoSimplify some Rust expression-evaluation code
Tom Tromey [Sun, 23 Jan 2022 19:48:38 +0000 (12:48 -0700)]
Simplify some Rust expression-evaluation code

A few Rust operations do a bit of work in their 'evaluate' functions
and then call another function -- but are also the only caller.  This
patch simplifies this code by removing the extra layer.

Tested on x86-64 Fedora 34.  I'm checking this in.

2 years agobfd: Partially revert commit 0e3839bde6f
H.J. Lu [Sun, 23 Jan 2022 18:40:46 +0000 (10:40 -0800)]
bfd: Partially revert commit 0e3839bde6f

Partially revert

commit 0e3839bde6f93e1e3eefce815be3636e3d81054d
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Sun Jan 23 07:29:27 2022 -0800

    bfd: Properly install library and header files

PR binutils/28807
* Makefile.am: Revert bfdlib_LTLIBRARIES and bfdinclude_HEADERS
changes.
* Makefile.in: Regenerate.

2 years agobfd: Properly install library and header files
H.J. Lu [Sun, 23 Jan 2022 15:29:27 +0000 (07:29 -0800)]
bfd: Properly install library and header files

Rename bfdlib_LTLIBRARIES and bfdinclude_HEADERS to lib_LTLIBRARIES and
include_HEADERS to fix the missing installed library and header files in
bfd caused by

commit bd32be01c997f686ab0b53f0640eaa0aeb61fbd3
Author: Mike Frysinger <vapier@gentoo.org>
Date:   Fri Dec 3 00:23:20 2021 -0500

    bfd: merge doc subdir up a level

PR binutils/28807
* Makefile.am (bfdlib_LTLIBRARIES): Renamed to ...
(lib_LTLIBRARIES): This.
(bfdinclude_HEADERS): Renamed to ...
(include_HEADERS): This.
* Makefile.in: Regenerate.
* doc/local.mk (install): Removed.

2 years agoRegenerate Makefile.in files with automake 1.15.1
H.J. Lu [Sun, 23 Jan 2022 14:59:20 +0000 (06:59 -0800)]
Regenerate Makefile.in files with automake 1.15.1

Regenerate Makefile.in files with the unmodified automake 1.15.1 to
remove

runstatedir = @runstatedir@

bfd/

* Makefile.in: Regenerate.

binutils/

* Makefile.in: Regenerate.

gas/

* Makefile.in: Regenerate.

gold/

* Makefile.in: Regenerate.
* testsuite/Makefile.in: Likewise.

gprof/

* Makefile.in: Regenerate.

ld/

* Makefile.in: Regenerate.

opcodes/

* Makefile.in: Regenerate.

2 years agoRegenerate configure files with autoconf 2.69
H.J. Lu [Sun, 23 Jan 2022 13:27:01 +0000 (05:27 -0800)]
Regenerate configure files with autoconf 2.69

Regenerate configure files with the unmodified autoconf 2.69 to remove

  --runstatedir=DIR       modifiable per-process data [LOCALSTATEDIR/run]

bfd/

* configure: Regenerate.

binutils/

* configure: Regenerate.

gas/

* configure: Regenerate.

gold/

* configure: Regenerate.

gprof/

* configure: Regenerate.

ld/

* configure: Regenerate.

opcodes/

* configure: Regenerate.

2 years agoAutomatic date update in version.in
GDB Administrator [Sun, 23 Jan 2022 00:00:12 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agobfd: merge doc subdir up a level
Mike Frysinger [Fri, 3 Dec 2021 05:23:20 +0000 (00:23 -0500)]
bfd: merge doc subdir up a level

This avoids a recursive make into the doc subdir and speeds up the
build slightly.  It also allows for more parallelism.

2 years agobfd: rename core.texi to corefile.texi
Mike Frysinger [Mon, 20 Dec 2021 07:05:31 +0000 (02:05 -0500)]
bfd: rename core.texi to corefile.texi

This is a generated file name from a correspondingly named C file.
Rename it to avoid unique build rules since there's no difference
to the generated manual.

2 years agobfd: replace doc header generation with pattern rules
Mike Frysinger [Mon, 20 Dec 2021 06:02:19 +0000 (01:02 -0500)]
bfd: replace doc header generation with pattern rules

This unifies boilerplate rules for most files with pattern rules.

2 years agoAllow inferring tmp_prefix from the dll name from a def file.
Martin Storsj? [Sat, 22 Jan 2022 14:31:22 +0000 (14:31 +0000)]
Allow inferring tmp_prefix from the dll name from a def file.

2 years agoAdjust default page sizes for haiku arm.
Alexander von Gluck IV [Sat, 22 Jan 2022 14:18:34 +0000 (14:18 +0000)]
Adjust default page sizes for haiku arm.

* configure.tgt (arm-haiku): Fix typo.
* emulparams/armelf_haiku.su (MAXPAGESIZE): Use the default value.
(COMMONPAGESIZE): Likewise.

2 years agoUpdate release makeing script with new release numbers
Nick Clifton [Sat, 22 Jan 2022 13:26:54 +0000 (13:26 +0000)]
Update release makeing script with new release numbers

2 years agoChange version number to 2.38.50 and regenerate files
Nick Clifton [Sat, 22 Jan 2022 12:39:28 +0000 (12:39 +0000)]
Change version number to 2.38.50 and regenerate files

2 years agoAdd markers for 2.38 branch
Nick Clifton [Sat, 22 Jan 2022 12:08:55 +0000 (12:08 +0000)]
Add markers for 2.38 branch

2 years agoRISC-V: create new frag after alignment.
Lifang Xia [Thu, 20 Jan 2022 02:42:55 +0000 (10:42 +0800)]
RISC-V: create new frag after alignment.

PR 28793:

The alignment may be removed in linker. We need to create new frag after
alignment to prevent the assembler from computing static offsets.

gas/
* config/tc-riscv.c (riscv_frag_align_code): Create new frag.

2 years agoAutomatic date update in version.in
GDB Administrator [Sat, 22 Jan 2022 00:00:12 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agogdb: include gdbsupport/buildargv.h in ser-mingw.c
Simon Marchi [Fri, 21 Jan 2022 16:33:29 +0000 (11:33 -0500)]
gdb: include gdbsupport/buildargv.h in ser-mingw.c

Fixes:

      CXX    ser-mingw.o
    /home/simark/src/binutils-gdb/gdb/ser-mingw.c: In function ‘int pipe_windows_open(serial*, const char*)’:
    /home/simark/src/binutils-gdb/gdb/ser-mingw.c:870:3: error: ‘gdb_argv’ was not declared in this scope
      870 |   gdb_argv argv (name);
          |   ^~~~~~~~

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=28802
Change-Id: I7f3e8ec5f9ca8582d587545fdf6b69901259f199

2 years agoUpdated Serbian translation for the ld sub-directory
Nick Clifton [Fri, 21 Jan 2022 15:42:18 +0000 (15:42 +0000)]
Updated Serbian translation for the ld sub-directory

2 years agogdb/doc: fill in two missing @r
Andrew Burgess [Fri, 21 Jan 2022 12:49:48 +0000 (12:49 +0000)]
gdb/doc: fill in two missing @r

I noticed two places in the docs where we appear to be missing @r.
makeinfo seems to do the correct things despite these being
missing (at least, I couldn't see any difference in the pdf or info
output), but it doesn't hurt to have the @r in place.

2 years agodrop old unused stamp-h.in file
Mike Frysinger [Fri, 21 Jan 2022 04:09:42 +0000 (23:09 -0500)]
drop old unused stamp-h.in file

This was needed by ancient versions of automake, but that hasn't been
the case since at least automake-1.5, so punt this from the tree.

2 years agogdbsupport/gdb_regex.cc: replace defs.h include with common-defs.h
Simon Marchi [Fri, 21 Jan 2022 03:57:11 +0000 (22:57 -0500)]
gdbsupport/gdb_regex.cc: replace defs.h include with common-defs.h

This was forgotten when gdb_regex was moved from gdb to gdbsupport.

Change-Id: I73b446f71861cabbf7afdb7408ef9d59fa64b804

2 years agoAutomatic date update in version.in
GDB Administrator [Fri, 21 Jan 2022 00:00:15 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agoAvoid bad breakpoints with --gc-sections
Tom Tromey [Thu, 13 Jan 2022 16:48:18 +0000 (09:48 -0700)]
Avoid bad breakpoints with --gc-sections

We found a case where --gc-sections can cause gdb to set an invalid
breakpoint.  In the included test case, gdb will set a breakpoint with
two locations, one of which is 0x0.

The code in lnp_state_machine::check_line_address is intended to
filter out this sort of problem, but in this case, the entire CU is
empty, causing unrelocated_lowpc==0x0 -- which circumvents the check.

It seems to me that if a CU is empty like this, then it is ok to
simply ignore the line table, as there won't be any locations anyway.

2 years agoAutomatic date update in version.in
GDB Administrator [Thu, 20 Jan 2022 00:00:18 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agoAdd `set print array-indexes' tests for C/C++ arrays
Maciej W. Rozycki [Wed, 19 Jan 2022 21:55:10 +0000 (21:55 +0000)]
Add `set print array-indexes' tests for C/C++ arrays

Add `set print array-indexes' tests for C/C++ arrays, complementing one
for Fortran arrays.

2 years agoRespect `set print array-indexes' with Fortran arrays
Maciej W. Rozycki [Wed, 19 Jan 2022 21:55:10 +0000 (21:55 +0000)]
Respect `set print array-indexes' with Fortran arrays

Add `set print array-indexes' handling for Fortran arrays.  Currently
the setting is ignored and indices are never shown.

Keep track of the most recent index handled so that any outstanding
repeated elements printed when the limit set by `set print elements' is
hit have the correct index shown.

Output now looks like:

(gdb) set print array-indexes on
(gdb) print array_1d
$1 = ((-2) = 1, (-1) = 1, (0) = 1, (1) = 1, (2) = 1)
(gdb) set print repeats 4
(gdb) set print elements 12
(gdb) print array_2d
$2 = ((-2) = ((-2) = 2, <repeats 5 times>) (-1) = ((-2) = 2, <repeats 5 times>) (0) = ((-2) = 2, (-1) = 2, ...) ...)
(gdb)

for a 5-element vector and a 5 by 5 array filled with the value of 2.

2 years agoAdd `set print repeats' tests for C/C++ arrays
Maciej W. Rozycki [Wed, 19 Jan 2022 21:55:10 +0000 (21:55 +0000)]
Add `set print repeats' tests for C/C++ arrays

Add `set print repeats' tests for C/C++ arrays, complementing one for
Fortran arrays and covering the different interpretation of the `set
print elements' setting in particular where the per-dimension count of
the elements handled is matched against the trigger rather than the
total element count as with Fortran arrays.

2 years agoRespect `set print repeats' with Fortran arrays
Maciej W. Rozycki [Wed, 19 Jan 2022 21:55:10 +0000 (21:55 +0000)]
Respect `set print repeats' with Fortran arrays

Implement `set print repeats' handling for Fortran arrays.  Currently
the setting is ignored and always treated as if no limit was set.

Unlike the generic array walker implemented decades ago the Fortran one
is a proper C++ class.  Rather than trying to mimic the old walker then,
which turned out a bit of a challenge where interacting with the `set
print elements' setting, write it entirely from scratch, by adding an
extra specialization handler method for processing dimensions other than
the innermost one and letting the specialization class call the `walk_1'
method from the handler as it sees fit.  This way repeats can be tracked
and the next inner dimension recursed into as a need arises only, or
unconditionally in the base class.

Keep track of the dimension number being handled in the class rather as
a parameter to the walker so that it does not have to be passed across
by the specialization class.

Use per-dimension element count tracking, needed to terminate processing
early when the limit set by `set print elements' is hit.  This requires
extra care too where the limit triggers exactly where another element
that is a subarray begins.  In that case rather than recursing we need
to terminate processing or lone `(...)' would be printed.  Additionally
if the skipped element is the last one in the current dimension we need
to print `...' by hand, because `continue_walking' won't print it at the
upper level, because it can see the last element has already been taken
care of.

Preserve the existing semantics of `set print elements' where the total
count of the elements handled is matched against the trigger level which
is unlike with the C/C++ array printer where the per-dimension element
count is used instead.

Output now looks like:

(gdb) set print repeats 4
(gdb) print array_2d
$1 = ((2, <repeats 5 times>) <repeats 5 times>)
(gdb) set print elements 12
(gdb) print array_2d
$2 = ((2, <repeats 5 times>) (2, <repeats 5 times>) (2, 2, ...) ...)
(gdb)

for a 5 by 5 array filled with the value of 2.

Amend existing test cases accordingly that rely on the current incorrect
behavior and explicitly request that there be no limit for printing
repeated elements there.

Add suitable test cases as well covering sliced arrays in particular.

Co-Authored-By: Andrew Burgess <andrew.burgess@embecosm.com>
2 years agofbsd-nat: Add include for gdb_argv.
John Baldwin [Wed, 19 Jan 2022 16:30:38 +0000 (08:30 -0800)]
fbsd-nat: Add include for gdb_argv.

2 years agoPowerPC64 DT_RELR ELFv1
Alan Modra [Wed, 19 Jan 2022 02:49:51 +0000 (13:19 +1030)]
PowerPC64 DT_RELR ELFv1

More fun with R_PPC64_NONE found in .opd.  Fixed by the
allocate_dynrelocs and ppc64_elf_size_dynamic_sections changes, and
since we are doing ifunc, opd and SYMBOL_REFERENCES_LOCAL tests later,
don't duplicate that work in check_relocs.

* elf64-ppc.c (ppc64_elf_check_relocs): Remove opd and ifunc
conditions for rel_count.
(dec_dynrel_count): Likewise.
(allocate_dynrelocs): Test for opd and ifunc when allocating
relative relocs.
(ppc64_elf_size_dynamic_sections): Likewise.

2 years agoPowerPC64 DT_RELR local PLT
Alan Modra [Wed, 19 Jan 2022 02:58:04 +0000 (13:28 +1030)]
PowerPC64 DT_RELR local PLT

Similarly to the local GOT case.

* elf64-ppc.c (ppc64_elf_size_dynamic_sections): Don't allocate
space for PLT relocs against local syms when enable_dt_relr.

2 years agoPowerPC64 DT_RELR local GOT
Alan Modra [Wed, 19 Jan 2022 00:23:53 +0000 (10:53 +1030)]
PowerPC64 DT_RELR local GOT

Fixes another case where we end up with superfluous R_PPC64_NONE.

* elf64-ppc.c (ppc64_elf_size_dynamic_sections): Don't allocate
space for GOT relocs against non-TLS local syms when enable_dt_relr.
(ppc64_elf_layout_multitoc): Likewise.

2 years agoAutomatic date update in version.in
GDB Administrator [Wed, 19 Jan 2022 00:00:10 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agoRe: PowerPC64 DT_RELR
Alan Modra [Tue, 18 Jan 2022 23:05:00 +0000 (09:35 +1030)]
Re: PowerPC64 DT_RELR

HJ: "There are 238 R_PPC64_NONEs in libc.so.6 alone."
Indeed, let's make them go away.  I had the SYMBOL_REFERENCES_LOCAL
test in the wrong place.  check_relocs is too early to know whether a
symbol is dynamic in a shared library.  Lots of glibc symbols are made
local by version script, but that doesn't happen until
size_dynamic_sections.

* elf64-ppc.c (ppc64_elf_check_relocs): Don't count relative relocs
here depending on SYMBOL_REFERENCES_LOCAL.
(dec_dynrel_count): Likewise.
(allocate_dynrelocs): Do so here instead.

2 years agoFix the remote-sim.c build
Tom Tromey [Tue, 18 Jan 2022 21:12:23 +0000 (14:12 -0700)]
Fix the remote-sim.c build

My earlier patch to move gdb_argv broke the remote-sim.c build.  This
patch fixes the bug.  I'm checking it in.

2 years agogdbserver: introduce remote_debug_printf
Simon Marchi [Mon, 17 Jan 2022 02:16:02 +0000 (21:16 -0500)]
gdbserver: introduce remote_debug_printf

Add remote_debug_printf, and use it for all debug messages controlled by
remote_debug.

Change remote_debug to be a bool, which is trivial in this case.

Change-Id: I90de13cb892faec3830047b571661822b126d6e8

2 years agogdbserver: introduce threads_debug_printf, THREADS_SCOPED_DEBUG_ENTER_EXIT
Simon Marchi [Mon, 17 Jan 2022 02:27:58 +0000 (21:27 -0500)]
gdbserver: introduce threads_debug_printf, THREADS_SCOPED_DEBUG_ENTER_EXIT

Add the threads_debug_printf and THREADS_SCOPED_DEBUG_ENTER_EXIT, which
use the logging infrastructure from gdbsupport/common-debug.h.  Replace
all debug_print uses that are predicated by debug_threads with
threads_dethreads_debug_printf.  Replace uses of the debug_enter and
debug_exit macros with THREADS_SCOPED_DEBUG_ENTER_EXIT, which serves
essentially the same purpose, but allows showing what comes between the
enter and the exit in an indented form.

Note that "threads" debug is currently used for a bit of everything in
GDBserver, not only threads related stuff.  It should ideally be cleaned
up and separated logically as is done in GDB, but that's out of the
scope of this patch.

Change-Id: I2d4546464462cb4c16f7f1168c5cec5a89f2289a

2 years agogdbserver: turn debug_threads into a boolean
Simon Marchi [Mon, 17 Jan 2022 02:21:24 +0000 (21:21 -0500)]
gdbserver: turn debug_threads into a boolean

debug_threads is always used as a boolean.  Except in ax.cc and
tracepoint.cc.  These files have their own macros that use
debug_threads, and have a concept of verbosity level.  But they both
have a single level, so it's just a boolean in the end.

Remove this concept of level.  If we ever want to re-introduce it, I
think it will be better implemented in a more common location.

Change debug_threads to bool and adjust some users that were treating it
as an int.

Change-Id: I137f596eaf763a08c977dd74417969cedfee9ecf

2 years agoSimplify Ada catchpoints
Tom Tromey [Thu, 13 Jan 2022 01:11:53 +0000 (18:11 -0700)]
Simplify Ada catchpoints

All the Ada catchpoints use the same breakpoint_ops contents, because
the catchpoint itself records its kind.  This patch simplifies the
code by removing the redundant ops structures.

2 years agoMove "catch exec" to a new file
Tom Tromey [Wed, 12 Jan 2022 23:13:23 +0000 (16:13 -0700)]
Move "catch exec" to a new file

The "catch exec" code is reasonably self-contained, and so this patch
moves it out of breakpoint.c (the second largest source file in gdb)
and into a new file, break-catch-exec.c.

2 years agoMove "catch fork" to a new file
Tom Tromey [Wed, 12 Jan 2022 23:03:33 +0000 (16:03 -0700)]
Move "catch fork" to a new file

The "catch fork" code is reasonably self-contained, and so this patch
moves it out of breakpoint.c (the second largest source file in gdb)
and into a new file, break-catch-fork.c.

2 years agoUnify "catch fork" and "catch vfork"
Tom Tromey [Wed, 12 Jan 2022 22:55:50 +0000 (15:55 -0700)]
Unify "catch fork" and "catch vfork"

I noticed that "catch fork" and "catch vfork" are nearly identical.
This patch simplifies the code by unifying these two cases.

2 years agoMove gdb_regex to gdbsupport
Tom Tromey [Sat, 1 Jan 2022 18:31:47 +0000 (11:31 -0700)]
Move gdb_regex to gdbsupport

This moves the gdb_regex convenience class to gdbsupport.

2 years agoIntroduce gdb-hashtab module in gdbsupport
Tom Tromey [Tue, 21 Dec 2021 23:48:38 +0000 (16:48 -0700)]
Introduce gdb-hashtab module in gdbsupport

gdb has some extensions and helpers for working with the libiberty
hash table.  This patch consolidates these and moves them to
gdbsupport.

2 years agoMove gdb obstack code to gdbsupport
Tom Tromey [Tue, 21 Dec 2021 23:38:32 +0000 (16:38 -0700)]
Move gdb obstack code to gdbsupport

This moves the gdb-specific obstack code -- both extensions like
obconcat and obstack_strdup, and things like auto_obstack -- to
gdbsupport.

2 years agoMove gdb_argv to gdbsupport
Tom Tromey [Wed, 26 Feb 2020 01:14:19 +0000 (18:14 -0700)]
Move gdb_argv to gdbsupport

This moves the gdb_argv class to a new header in gdbsupport.

2 years agoSimplify event_location_probe
Tom Tromey [Fri, 14 Jan 2022 16:06:11 +0000 (09:06 -0700)]
Simplify event_location_probe

event_location_probe currently stores two strings, but really only
needs one.  This patch simplifies it and removes some unnecessary
copies as well.

2 years agoUse std::string in event_location
Tom Tromey [Fri, 14 Jan 2022 15:53:35 +0000 (08:53 -0700)]
Use std::string in event_location

This changes event_location to use std::string, removing some manual
memory management, and an unnecessary string copy.

2 years agoSplit event_location into subclasses
Tom Tromey [Fri, 14 Jan 2022 15:29:42 +0000 (08:29 -0700)]
Split event_location into subclasses

event_location uses the old C-style discriminated union approach.
However, it's better to use subclassing, as this makes the code
clearer and removes some chances for error.  This also enables future
cleanups to avoid manual memory management and copies.

2 years agoRemove EL_* macros from location.c
Tom Tromey [Fri, 14 Jan 2022 14:33:33 +0000 (07:33 -0700)]
Remove EL_* macros from location.c

This patch removes the old-style EL_* macros from location.c.  This
cleans up the code by itself, IMO, but also enables further cleanups
in subsequent patches.

2 years agoBoolify explicit_to_string_internal
Tom Tromey [Fri, 14 Jan 2022 14:26:12 +0000 (07:26 -0700)]
Boolify explicit_to_string_internal

This changes explicit_to_string_internal to use 'bool' rather than
'int'.

2 years agoRemove a use of xfree in location.c
Tom Tromey [Fri, 14 Jan 2022 14:24:18 +0000 (07:24 -0700)]
Remove a use of xfree in location.c

This small cleanup removes a use of xfree from location.c, by
switching to unique_xmalloc_ptr.  One function is only used in
location.c, so it is made static.  And, another function is changed to
avoid a copy.

2 years agogdb: use ptid_t::to_string instead of target_pid_to_str in debug statements
Simon Marchi [Sun, 16 Jan 2022 04:45:06 +0000 (23:45 -0500)]
gdb: use ptid_t::to_string instead of target_pid_to_str in debug statements

Same idea as 0fab79556484 ("gdb: use ptid_t::to_string in infrun debug
messages"), but throughout GDB.

Change-Id: I62ba36eaef29935316d7187b9b13d7b88491acc1

2 years agogdb: preserve `|` in connection details string
Andrew Burgess [Wed, 12 Jan 2022 17:13:38 +0000 (17:13 +0000)]
gdb: preserve `|` in connection details string

Consider this GDB session:

  $ gdb -q
  (gdb) target remote  | gdbserver - ~/tmp/hello.x
  Remote debugging using | gdbserver - ~/tmp/hello.x
  ... snip ...
  (gdb) info connections
    Num  What                              Description
  * 1    remote gdbserver - ~/tmp/hello.x  Remote target using gdb-specific protocol
  (gdb) python conn = gdb.selected_inferior().connection
  (gdb) python print(conn.details)
  gdbserver - ~/tmp/hello.x
  (gdb)

I think there are two things wrong here, first in the "What" column of
the 'info connections' output, I think the text should be:

  remote | gdbserver - ~/tmp/hello.x

to correctly show the user how the connection was established.  And in
a similar fashion, I think that the `details` string of the
gdb.TargetConnection object should be:

  | gdbserver - ~/tmp/hello.x

This commit makes this change.  Currently the '|' is detected and
removed in gdb/serial.c.  The string passed to the pipe_ops
structure (from gdb/ser-pipe.c), doesn't then, contain the `|`, this
is instead implied by the fact that it is a pipes based implementation
of the serial_ops interface.

After this commit we still detect the `|` in gdb/serial.c, but we now
store the full string (including the `|`) in the serial::name member
variable.

For pipe based serial connections, this name is only used for
displaying the two fields I mention above, and in pipe_open (from
gdb/ser-pipe.c), and in pipe_open, we now know to skip over the `|`.

The benefit I see from this change is that GDB's output now more
accurately reflects the commands used to start a target, thus making
it easier for a user to understand what is going on.

2 years agogdb: testsuite: print explicit test result for gdb.base/dfp-test.exp
Tiezhu Yang [Tue, 18 Jan 2022 07:17:57 +0000 (15:17 +0800)]
gdb: testsuite: print explicit test result for gdb.base/dfp-test.exp

In the current code, if decimal floating point is not supported for
this target, there is no binary file dfp-test, and also there is no
test result after execute the following commands:

  $ make check-gdb TESTS="gdb.base/dfp-test.exp"
  $ grep error gdb/testsuite/gdb.log
  /home/loongson/gdb.git/gdb/testsuite/gdb.base/dfp-test.c:39:1: error: decimal floating point not supported for this target
  [...]
  $ cat gdb/testsuite/gdb.sum
  [...]
  Running target unix
  Running /home/loongson/gdb.git/gdb/testsuite/gdb.base/dfp-test.exp ...

  === gdb Summary ===
  [...]

With this patch:

  $ make check-gdb TESTS="gdb.base/dfp-test.exp"
  $ cat gdb/testsuite/gdb.sum
  [...]
  Running target unix
  Running /home/loongson/gdb.git/gdb/testsuite/gdb.base/dfp-test.exp ...
  UNSUPPORTED: gdb.base/dfp-test.exp: decimal floating point not supported for this target.

  === gdb Summary ===

  # of unsupported tests 1
  [...]

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
2 years agobfd/elf64-ppc.c: fix clang -Wbitwise-instead-of-logical warning in ppc64_elf_check_in...
Simon Marchi [Sun, 16 Jan 2022 03:13:07 +0000 (22:13 -0500)]
bfd/elf64-ppc.c: fix clang -Wbitwise-instead-of-logical warning in ppc64_elf_check_init_fini

I see this error with clang-14:

      CC       elf64-ppc.lo
    /home/smarchi/src/binutils-gdb/bfd/elf64-ppc.c:13131:11: error: use of bitwise '&' with boolean operands [-Werror,-Wbitwise-instead-of-logical]
      return (check_pasted_section (info, ".init")
             ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Fix by replacing & with &&.  But given that the check_pasted_section
function has side-effects and we want to make sure both calls are made,
assign to temporary variables before evaluating the `&&`.

Change-Id: I849e1b2401bea5f4d8ef3ab9af99ba9e3ef42490

2 years agoPR28029, debuginfod tests
Alan Modra [Sun, 21 Nov 2021 23:46:01 +0000 (10:16 +1030)]
PR28029, debuginfod tests

binutils/NEWS says of the change in --process-links semantics:
  If other debug section display options are also enabled (eg
  --debug-dump=info) then the contents of matching sections in both the main
  file and the separate debuginfo file *will* be displayed.  This is because in
  most cases the debug section will only be present in one of the files.

Implying that debug info is dumped without --process-links.  Indeed
that appears to be the case for readelf.  This does the same for
objdump.

PR 28029
* objdump.c (dump_bfd): Do not exit early when !is_mainfile
&& !processlinks, instead just exclude non-debug output.
(dump_dwarf): Add is_mainfile parameter and pass to
dump_dwarf_section.
(dump_dwarf_section): Only display debug sections when
!is_mainfile and !process_links.

2 years agoCheck thin archive element file size against archive header
Alan Modra [Tue, 28 Dec 2021 12:21:55 +0000 (22:51 +1030)]
Check thin archive element file size against archive header

Makes it a little less likely for someone to break their thin archives.

* archive.c (_bfd_get_elt_at_filepos): Check thin archive
element file size.

2 years agolang_size_relro_segment tidy
Alan Modra [Fri, 14 Jan 2022 01:54:22 +0000 (12:24 +1030)]
lang_size_relro_segment tidy

This function has seen too many minimal change style edits.
No functional changes in this patch.

* ldlang.c (lang_size_relro_segment): Tidy.

2 years agoPowerPC64 DT_RELR
Alan Modra [Thu, 6 Jan 2022 21:19:16 +0000 (07:49 +1030)]
PowerPC64 DT_RELR

PowerPC64 takes a more traditional approach to DT_RELR than x86.  Count
relative relocs in check_relocs, allocate space for them and output in
the usual places but not doing so when enable_dt_relr.  DT_RELR is
sized in the existing ppc stub relaxation machinery, run via the
linker's ldemul_after_allocation hook.  DT_RELR is output in the same
function that writes ppc stubs, run via ldemul_finish.

This support should be considered experimental.

bfd/
* elf64-ppc.c (struct ppc_local_dyn_relocs): Renamed from
ppc_dyn_relocs.  Add rel_count field.  Update uses.
(struct ppc_dyn_relocs): New.  Replace all uses of elf_dyn_relocs.
(struct ppc_link_hash_table): Add relr_alloc, relr_count and
relr_addr.
(ppc64_elf_copy_indirect_symbol): Merge rel_count.
(ppc64_elf_check_relocs): Init rel_count for global and local syms.
(dec_dynrel_count): Change r_info param to reloc pointer.  Update
all callers.  Handle decrementing rel_count.
(allocate_got): Don't allocate space for relative relocs when
enable_dt_relr.
(allocate_dynrelocs): Likewise.
(ppc64_elf_size_dynamic_sections): Likewise.  Handle srelrdyn.
(ppc_build_one_stub): Don't emit relative relocs on .branch_lt.
(compare_relr_address, append_relr_off): New functions.
(got_and_plt_relr_for_local_syms, got_and_plt_relr): Likewise.
(ppc64_elf_size_stubs): Size .relr.syn.
(ppc64_elf_build_stubs): Emit .relr.dyn.
(build_global_entry_stubs_and_plt): Don't output relative relocs
when enable_dt_relr.
(write_plt_relocs_for_local_syms): Likewise.
(ppc64_elf_relocate_section): Likewise.
binutils/
* testsuite/lib/binutils-common.exp (supports_dt_relr): Add
powerpc64.
ld/
* emulparams/elf64ppc.sh: Source dt-relr.sh.
* testsuite/ld-elf/dt-relr-2b.d: Adjust for powerpc.
* testsuite/ld-elf/dt-relr-2c.d: Likewise.
* testsuite/ld-elf/dt-relr-2d.d: Likewise.
* testsuite/ld-elf/dt-relr-2e.d: Likewise.

2 years agotweak __ehdr_start visibility and flags for check_relocs
Alan Modra [Wed, 12 Jan 2022 13:12:23 +0000 (23:42 +1030)]
tweak __ehdr_start visibility and flags for check_relocs

bfd/
* elf-bfd.h (UNDEFWEAK_NO_DYNAMIC_RELOC): Test linker_def.
ld/
* ldelf.c (ldelf_before_allocation): Don't force __ehdr_start
local and hidden here..
* ldlang.c (lang_symbol_tweaks): ..do so here instead and set
def_regular and linker_def for check_relocs.  New function
extracted from lang_process.

2 years agoAutomatic date update in version.in
GDB Administrator [Tue, 18 Jan 2022 00:00:16 +0000 (00:00 +0000)]
Automatic date update in version.in

2 years agoUpdate the config.guess and config.sub files from the master repository and regenerat...
Nick Clifton [Mon, 17 Jan 2022 16:21:22 +0000 (16:21 +0000)]
Update the config.guess and config.sub files from the master repository and regenerate files.

2 years agoFix Z80 assembly failure.
Sergey Belyashov [Mon, 17 Jan 2022 13:00:17 +0000 (13:00 +0000)]
Fix Z80 assembly failure.

PR 28762
* app.c (do_scrub_chars): Correct handling when the symbol is not 'af'.

2 years agogdb/infrun: rename variable and move to more specific scope
Simon Marchi [Mon, 10 Jan 2022 21:40:54 +0000 (16:40 -0500)]
gdb/infrun: rename variable and move to more specific scope

Move the "started" variable to the scope it's needed, and rename it to
"step_over_started".

Change-Id: I56f3384dbd328f55198063bb855edda10f1492a3

2 years agox86: adjust struct instr_info field types
Jan Beulich [Mon, 17 Jan 2022 09:26:43 +0000 (10:26 +0100)]
x86: adjust struct instr_info field types

Now that this lives on the stack, let's have it be a little less
wasteful in terms of space. Switch boolean fields to "bool" (also when
this doesn't change their size) and also limit the widths of "rex",
"rex_used", "op_ad", and "op_index". Do a little bit of re-ordering as
well to limit the number of padding holes.

2 years agox86: drop index16 field
Jan Beulich [Mon, 17 Jan 2022 09:26:21 +0000 (10:26 +0100)]
x86: drop index16 field

There's a single use on a generally infrequently taken code path. Put
the necessary conditional there instead.

2 years agox86: drop most Intel syntax register name arrays
Jan Beulich [Mon, 17 Jan 2022 09:26:01 +0000 (10:26 +0100)]
x86: drop most Intel syntax register name arrays

By making use of, in particular, oappend_maybe_intel() there's no need
for this redundant set of static data.

2 years agox86: fold variables in memory operand index handling
Jan Beulich [Mon, 17 Jan 2022 09:25:40 +0000 (10:25 +0100)]
x86: fold variables in memory operand index handling

There's no real need for the pseudo-boolean "haveindex" or for separate
32-bit / 64-bit index pointers. Fold them into a single "indexes" and
set that uniformly to AT&T names, compensating by emitting the register
name via oappend_maybe_intel().

2 years agox86: constify disassembler static data
Jan Beulich [Mon, 17 Jan 2022 09:25:08 +0000 (10:25 +0100)]
x86: constify disassembler static data

Now that the code is intended to be largely thread-safe, we'd better not
have any writable static objects.

2 years agoAutomatic date update in version.in
GDB Administrator [Mon, 17 Jan 2022 00:00:25 +0000 (00:00 +0000)]
Automatic date update in version.in