From e9307449c314ac40a660e5774dd5175571cb2382 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Sun, 28 Nov 2021 14:25:36 -0500 Subject: [PATCH] sim: add checks to core headers to prevent incorrect common building Some of the core sim headers rely on the SIM_AC_OPTION_BITSIZE macro which can change the size of core types. Since these haven't been unified across ports, add checks to make sure they aren't accidentally included when building for all ports. This caught the sim-load file using poisoned headers that it didn't actually need. --- sim/common/sim-config.h | 3 +++ sim/common/sim-load.c | 3 +-- sim/common/sim-types.h | 4 ++++ 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/sim/common/sim-config.h b/sim/common/sim-config.h index aae08e71c58..88bdedf687d 100644 --- a/sim/common/sim-config.h +++ b/sim/common/sim-config.h @@ -23,6 +23,9 @@ #ifndef SIM_CONFIG_H #define SIM_CONFIG_H +#ifdef SIM_COMMON_BUILD +#error "This header is unusable in common builds due to reliance on SIM_AC_OPTION_BITSIZE" +#endif /* Host dependant: diff --git a/sim/common/sim-load.c b/sim/common/sim-load.c index 3ea50e7fbcd..ec5df418d9d 100644 --- a/sim/common/sim-load.c +++ b/sim/common/sim-load.c @@ -27,12 +27,11 @@ along with this program. If not, see . */ #include #include -#include "sim-basics.h" #include "bfd.h" -#include "sim-utils.h" #include "sim/callback.h" #include "sim/sim.h" +#include "sim-utils.h" static void eprintf (host_callback *, const char *, ...); static void xprintf (host_callback *, const char *, ...); diff --git a/sim/common/sim-types.h b/sim/common/sim-types.h index 8e1c9ada920..5fd0a1dcff2 100644 --- a/sim/common/sim-types.h +++ b/sim/common/sim-types.h @@ -23,6 +23,10 @@ #ifndef SIM_TYPES_H #define SIM_TYPES_H +#ifdef SIM_COMMON_BUILD +#error "This header is unusable in common builds due to reliance on SIM_AC_OPTION_BITSIZE" +#endif + #include /* INTEGER QUANTITIES: -- 2.30.2