mirror of
https://sourceware.org/git/binutils-gdb.git
synced 2025-01-30 12:44:10 +08:00
f559e52a8e
I noticed a few files double-included libcoff.h, and digging deeper I found that the PEI_HEADERS define is a sort of external include guard. This patch adds include guards to the few files in include/coff that were missing one, and then removes the PEI_HEADERS workaround and the redundant includes. I didn't see anything in these files that indicated that double-inclusion would be useful, so it seems to me that this approach is ok. Tested by rebuilding with --enable-targets=all. 2023-08-02 Tom Tromey <tromey@adacore.com> * pei-x86_64.c (PEI_HEADERS): Do not define. * pei-loongarch64.c (PEI_HEADERS): Do not define. * pei-aarch64.c (PEI_HEADERS): Do not define. * pe-x86_64.c (PEI_HEADERS): Do not define. * pe-aarch64.c (PEI_HEADERS): Do not define. * libpei.h (_LIBPEI_H): Add include guard. * coff-x86_64.c (PEI_HEADERS): Do not check. * coff-loongarch64.c (PEI_HEADERS): Do not check. * coff-aarch64.c (PEI_HEADERS): Do not check. include/ChangeLog 2023-08-02 Tom Tromey <tromey@adacore.com> * coff/x86_64.h (COFF_X86_64_H): Add include guard. * coff/loongarch64.h (COFF_LOONGARCH64_H): Add include guard. * coff/aarch64.h (COFF_AARCH64_H): Add include guard. |
||
---|---|---|
.. | ||
aarch64.h | ||
alpha.h | ||
arm.h | ||
ChangeLog-0415 | ||
ChangeLog-9103 | ||
ecoff.h | ||
external.h | ||
i386.h | ||
ia64.h | ||
internal.h | ||
loongarch64.h | ||
mcore.h | ||
mips.h | ||
msdos.h | ||
pe.h | ||
rs6k64.h | ||
rs6000.h | ||
sh.h | ||
sym.h | ||
symconst.h | ||
ti.h | ||
tic4x.h | ||
tic30.h | ||
tic54x.h | ||
x86_64.h | ||
xcoff.h | ||
z8k.h | ||
z80.h |