~ubuntu-support-team/binutils/+git/binutils-gdb:users/aburgess/try-mips-disasm-styling

Last commit made on 2022-11-03
Get this branch:
git clone -b users/aburgess/try-mips-disasm-styling https://git.launchpad.net/~ubuntu-support-team/binutils/+git/binutils-gdb

Branch merges

Branch information

Name:
users/aburgess/try-mips-disasm-styling
Repository:
lp:~ubuntu-support-team/binutils/+git/binutils-gdb

Recent commits

ca81edb... by Andrew Burgess <email address hidden>

libopcodes/mips: add support for disassembler styling

This commit adds disassembler styling support for MIPS. After this
commit objdump and GDB will style disassembler output.

This is a pretty straight forward change, we switch to use the
disassemble_info::fprintf_styled_func callback, and pass an
appropriate style through as needed. No additional tricks were
needed (compared to say i386, or ARM).

Tested by running all of the objdump commands used by the gas
testsuite and manually inspecting the styled output, everything looks
reasonable, though I'm not a MIPS expert, so it is possible that I've
missed some corner cases. Worst case though is that something will be
styled incorrectly, the actual content should be unchanged.

All the gas, ld, and binutils tests still pass for me.

e53fbf5... by Andrew Burgess <email address hidden>

opcodes/mips: use .word/.short for undefined instructions

While working on disassembler styling for MIPS, I noticed that
undefined instructions are printed by the disassembler as raw number
with no assembler directive prefix (e.g. without .word or .short).

I think adding something like .word, or .short, helps to make it
clearer the size of the value that is being displayed, and is inline
with what many of the other libopcode disassemblers do.

In this commit I've added the .word and .short directives, and updated
all the tests that I spotted that failed as a result.

78cd918... by Nick Clifton <email address hidden>

AVR Linker: Allow the start of the data region to be specified on the linker command line. [Fix PR number in ChangeLog entry]

 PR 29741
 * scripttempl/avr.sc (__DATA_REGION_ORIGIN__): Define. If a value
 has not been provided on the command line then use DATA_ORIGIN.
 (MEMORY): Use __DATA_REGION_ORIGIN__ as the start of the data region.

fa6895a... by Nick Clifton <email address hidden>

AVR Linker: Allow the start of the data region to specified on the command line.

 PR 29471
 * scripttempl/avr.sc (__DATA_REGION_ORIGIN__): Define. If a value
 has not been provided on the command line then use DATA_ORIGIN.
 (MEMORY): Use __DATA_REGION_ORIGIN__ as the start of the data region.

a1af8f4... by Mike Frysinger

sim: move common flags to default AM_CPPFLAGS

Since all host files we compile use these settings, move them out of
libcommon.a and into the default AM_CPPFLAGS. This has the effect of
dropping the custom per-target automake rules. Currently it saves us
~150 lines, but since it's about ~8 lines per object, the overhead
will increase quite a bit as we merge more files into a single build.

This also changes the object output names, so we have to tweak the
rules that were pulling in the common objects when linking.

46a1e1f... by Mike Frysinger

sim: merge gnulib logic into the top-level

We aren't using this just yet, but we will, so make it available to
building of common sim files.

ca7b34c... by Mike Frysinger

sim: common: remove unused include paths

A bunch of these paths don't include any headers, and most likely
never will, so there's no real need to keep them. This will let
us harmonize paths with the top-level Makefile more easily, which
will in turn make it easier to move more compile steps there.

ae1c45e... by GDB Administrator <email address hidden>

Automatic date update in version.in

7eb96b2... by Christophe Lyon <email address hidden>

arm: PR 29739 Fix typo where ';' should not have been replaced with '@'

';' does not always indicate the start of a comment, and commit
8cb6e17571f3fb66ccd4fa19f881602542cd06fc incorrectly replaced 3
instances of ';' with '@' in expected diagnostics, leading to tests
failures.

This patch restores the original ';' as needed in these testcases.

Fixes bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29739

fde7c6b... by Mike Frysinger

sim: split CPPFLAGS between build & host

In order to merge more common/ files into the top-level, we need to
add more host flags to CPPFLAGS, and that conflicts with our current
use with build-time tools. So split them apart like we do with all
other build flags to avoid the issue.