Fix compile time warning building aout targeted architectures.
authorGunther Nikl <gnikl@users.sourceforge.net>
Fri, 12 Jan 2018 13:12:17 +0000 (13:12 +0000)
committerNick Clifton <nickc@redhat.com>
Fri, 12 Jan 2018 13:12:17 +0000 (13:12 +0000)
commitb5b62006522c6d8d42028754bb1be165f00fe210
tree09f40f43d8fdc20264a7313a4ce0f6cb0195bc5c
parent5ae5d3fd10c4bb520d767b25c59bf86db764adef
Fix compile time warning building aout targeted architectures.

Occasionally I build an out-of-tree a.out target (m68k-amigaos). After
a system upgrade which included a newer compiler (clang 4) the build
produces warnings like this:

  warning: macro expansion producing 'defined' has undefined behavior
  [-Wexpansion-to-defined]

This is caused by the macro gas/config/aout_gnu.h:USE_EXTENDED_RELOC.
Since it is in a header file, the warning triggers for several files.
I am unsure what solution is preferable, thus I am suggesting two
patches:

  a) keep the offending macro but define it explicitly to 0 and 1
  b) replace the macro usage with its value where it is used.

Either patch removes the warning for clang. I did not check with a
recent GCC.

* gas/config/aout_gnu.h (USE_EXTENDED_RELOC): Explicitly
define to 0 and 1. Remove a dangling reference to "AMD 29000"
in a comment.
gas/ChangeLog
gas/config/aout_gnu.h