diff options
| author | Richard Earnshaw <rearnsha@arm.com> | 2025-09-01 16:52:54 +0100 |
|---|---|---|
| committer | Richard Earnshaw <rearnsha@arm.com> | 2025-09-02 16:12:12 +0100 |
| commit | 8132afd2de85bddf4a9ec56a693b631751495c5b (patch) | |
| tree | 652f5573c5c98937162739cfe13ce72811a7c12b /gdb/common/preprocessor.h | |
| parent | bd17933c5a001f636b2b7723a3758e98dbb78941 (diff) | |
| download | binutils-8132afd2de85bddf4a9ec56a693b631751495c5b.zip binutils-8132afd2de85bddf4a9ec56a693b631751495c5b.tar.gz binutils-8132afd2de85bddf4a9ec56a693b631751495c5b.tar.bz2 | |
arm: rework fix for PR gas/15273
PR gas/15273
The apparent intent of the original code added in
https://sourceware.org/pipermail/binutils/2012-August/078044.html was
to emit an error message if the instruction was obsolete and only to
emit a deprecation warning if an error hadn't already been emitted.
However, when the insn has not yet been obsoleted, the code would
generate a warning with a (null) message body if the selected CPU was
'any'.
The previous fix for this bug was to remove the support for the
obsoletion message entirely, which was probably the wrong approach. A
better fix is to only call check_obsolete if obs_msg is non-null; we
can use this as a proxy for the instruction not being obsolete on any
architecture.
While we are here, fix an incorrect capitalization in the deprecation
message ('This' to 'this').
Diffstat (limited to 'gdb/common/preprocessor.h')
0 files changed, 0 insertions, 0 deletions