From 54c47dfb68a5e2cfde5fec7224b0f92b12edc42c Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Tue, 15 Jun 2021 22:37:38 -0400 Subject: [PATCH] sim: ppc: rename inline defines to match common code Use the same basic names as the common sim inline logic so we can merge the two. We don't do that here, just prepare for it. The common code seems to be based on the ppc version but with slightly different names as it was cleaned up & generalized. I *think* these concepts are the same, so binding them together is OK, but maybe I'm misreading them. If so, can always tweak them later. REVEAL_MODULE -> H_REVEALS_MODULE INLINE_MODULE -> C_REVEALS_MODULE --- sim/ppc/ChangeLog | 13 +++++++++++++ sim/ppc/configure | 10 +++++----- sim/ppc/configure.ac | 10 +++++----- sim/ppc/inline.h | 42 +++++++++++++++++++++--------------------- sim/ppc/options.c | 8 ++++---- sim/ppc/std-config.h | 33 ++++++++++++++++----------------- 6 files changed, 64 insertions(+), 52 deletions(-) diff --git a/sim/ppc/ChangeLog b/sim/ppc/ChangeLog index 156e57a668f..ec248cb48dd 100644 --- a/sim/ppc/ChangeLog +++ b/sim/ppc/ChangeLog @@ -1,3 +1,16 @@ +2021-06-19 Mike Frysinger + + * configure.ac: Replace ALL_INLINE with ALL_C_INLINE and + PSIM_INLINE_LOCALS with INLINE_LOCALS. + * inline.h: Likewise. + * options.c: Likewise. + * std-config.h: Likewise. Include sim-inline.h explicitly. + (REVEAL_MODULE): Define to H_REVEALS_MODULE. + (INLINE_MODULE): Define to C_REVEALS_MODULE. + (PSIM_INLINE_LOCALS): Delete. + (ALL_INLINE): Delete. + * configure: Regenerate. + 2021-06-19 Mike Frysinger * Makefile.in: Move intl vars to ../arch-subdir.mk.in. diff --git a/sim/ppc/configure b/sim/ppc/configure index b1d2ca1491d..6228d17ddea 100755 --- a/sim/ppc/configure +++ b/sim/ppc/configure @@ -3311,15 +3311,15 @@ if test "${enable_sim_inline+set}" = set; then : case "$enableval" in no) sim_inline="-DDEFAULT_INLINE=0";; 0) sim_inline="-DDEFAULT_INLINE=0";; - yes | 2) sim_inline="-DDEFAULT_INLINE=ALL_INLINE";; - 1) sim_inline="-DDEFAULT_INLINE=PSIM_INLINE_LOCALS";; + yes | 2) sim_inline="-DDEFAULT_INLINE=ALL_C_INLINE";; + 1) sim_inline="-DDEFAULT_INLINE=INLINE_LOCALS";; *) for x in `echo "$enableval" | sed -e "s/,/ /g"`; do new_flag="" case "$x" in *_INLINE=*) new_flag="-D$x";; *=*) new_flag=`echo "$x" | sed -e "s/=/_INLINE=/" -e "s/^/-D/"`;; - *_INLINE) new_flag="-D$x=ALL_INLINE";; - *) new_flag="-D$x""_INLINE=ALL_INLINE";; + *_INLINE) new_flag="-D$x=ALL_C_INLINE";; + *) new_flag="-D$x""_INLINE=ALL_C_INLINE";; esac if test x"$sim_inline" = x""; then sim_inline="$new_flag" @@ -3333,7 +3333,7 @@ if test x"$silent" != x"yes" && test x"$sim_inline" != x""; then fi else if test x"$GCC" != ""; then - sim_inline="-DDEFAULT_INLINE=PSIM_INLINE_LOCALS" + sim_inline="-DDEFAULT_INLINE=INLINE_LOCALS" if test x"$silent" != x"yes"; then echo "Setting inline flags = $sim_inline" 6>&1 fi diff --git a/sim/ppc/configure.ac b/sim/ppc/configure.ac index 58c68a110f9..19dbd77bda9 100644 --- a/sim/ppc/configure.ac +++ b/sim/ppc/configure.ac @@ -275,15 +275,15 @@ AC_ARG_ENABLE(sim-inline, case "$enableval" in no) sim_inline="-DDEFAULT_INLINE=0";; 0) sim_inline="-DDEFAULT_INLINE=0";; - yes | 2) sim_inline="-DDEFAULT_INLINE=ALL_INLINE";; - 1) sim_inline="-DDEFAULT_INLINE=PSIM_INLINE_LOCALS";; + yes | 2) sim_inline="-DDEFAULT_INLINE=ALL_C_INLINE";; + 1) sim_inline="-DDEFAULT_INLINE=INLINE_LOCALS";; *) for x in `echo "$enableval" | sed -e "s/,/ /g"`; do new_flag="" case "$x" in *_INLINE=*) new_flag="-D$x";; *=*) new_flag=`echo "$x" | sed -e "s/=/_INLINE=/" -e "s/^/-D/"`;; - *_INLINE) new_flag="-D$x=ALL_INLINE";; - *) new_flag="-D$x""_INLINE=ALL_INLINE";; + *_INLINE) new_flag="-D$x=ALL_C_INLINE";; + *) new_flag="-D$x""_INLINE=ALL_C_INLINE";; esac if test x"$sim_inline" = x""; then sim_inline="$new_flag" @@ -295,7 +295,7 @@ esac if test x"$silent" != x"yes" && test x"$sim_inline" != x""; then echo "Setting inline flags = $sim_inline" 6>&1 fi],[if test x"$GCC" != ""; then - sim_inline="-DDEFAULT_INLINE=PSIM_INLINE_LOCALS" + sim_inline="-DDEFAULT_INLINE=INLINE_LOCALS" if test x"$silent" != x"yes"; then echo "Setting inline flags = $sim_inline" 6>&1 fi diff --git a/sim/ppc/inline.h b/sim/ppc/inline.h index 5fb108f09bb..da0bd5cf7eb 100644 --- a/sim/ppc/inline.h +++ b/sim/ppc/inline.h @@ -41,7 +41,7 @@ # define EXTERN_PSIM_ENDIAN(TYPE) TYPE #endif -#if (SIM_ENDIAN_INLINE & PSIM_INLINE_LOCALS) +#if (SIM_ENDIAN_INLINE & INLINE_LOCALS) # define STATIC_INLINE_PSIM_ENDIAN(TYPE) static INLINE TYPE #else # define STATIC_INLINE_PSIM_ENDIAN(TYPE) static TYPE @@ -63,7 +63,7 @@ # define EXTERN_BITS(TYPE) TYPE #endif -#if (BITS_INLINE & PSIM_INLINE_LOCALS) +#if (BITS_INLINE & INLINE_LOCALS) # define STATIC_INLINE_BITS(TYPE) static INLINE TYPE #else # define STATIC_INLINE_BITS(TYPE) static TYPE @@ -85,7 +85,7 @@ # define EXTERN_CORE(TYPE) TYPE #endif -#if (CORE_INLINE & PSIM_INLINE_LOCALS) +#if (CORE_INLINE & INLINE_LOCALS) # define STATIC_INLINE_CORE(TYPE) static INLINE TYPE #else # define STATIC_INLINE_CORE(TYPE) static TYPE @@ -107,7 +107,7 @@ # define EXTERN_VM(TYPE) TYPE #endif -#if (VM_INLINE & PSIM_INLINE_LOCALS) +#if (VM_INLINE & INLINE_LOCALS) # define STATIC_INLINE_VM(TYPE) static INLINE TYPE #else # define STATIC_INLINE_VM(TYPE) static TYPE @@ -129,7 +129,7 @@ # define EXTERN_CPU(TYPE) TYPE #endif -#if (CPU_INLINE & PSIM_INLINE_LOCALS) +#if (CPU_INLINE & INLINE_LOCALS) # define STATIC_INLINE_CPU(TYPE) static INLINE TYPE #else # define STATIC_INLINE_CPU(TYPE) static TYPE @@ -151,7 +151,7 @@ # define EXTERN_MODEL(TYPE) TYPE #endif -#if (MODEL_INLINE & PSIM_INLINE_LOCALS) +#if (MODEL_INLINE & INLINE_LOCALS) # define STATIC_INLINE_MODEL(TYPE) static INLINE TYPE #else # define STATIC_INLINE_MODEL(TYPE) static TYPE @@ -173,7 +173,7 @@ # define EXTERN_EVENTS(TYPE) TYPE #endif -#if (EVENTS_INLINE & PSIM_INLINE_LOCALS) +#if (EVENTS_INLINE & INLINE_LOCALS) # define STATIC_INLINE_EVENTS(TYPE) static INLINE TYPE #else # define STATIC_INLINE_EVENTS(TYPE) static TYPE @@ -195,7 +195,7 @@ # define EXTERN_MON(TYPE) TYPE #endif -#if (MON_INLINE & PSIM_INLINE_LOCALS) +#if (MON_INLINE & INLINE_LOCALS) # define STATIC_INLINE_MON(TYPE) static INLINE TYPE #else # define STATIC_INLINE_MON(TYPE) static TYPE @@ -217,7 +217,7 @@ # define EXTERN_REGISTERS(TYPE) TYPE #endif -#if (REGISTERS_INLINE & PSIM_INLINE_LOCALS) +#if (REGISTERS_INLINE & INLINE_LOCALS) # define STATIC_INLINE_REGISTERS(TYPE) static INLINE TYPE #else # define STATIC_INLINE_REGISTERS(TYPE) static TYPE @@ -239,7 +239,7 @@ # define EXTERN_INTERRUPTS(TYPE) TYPE #endif -#if (INTERRUPTS_INLINE & PSIM_INLINE_LOCALS) +#if (INTERRUPTS_INLINE & INLINE_LOCALS) # define STATIC_INLINE_INTERRUPTS(TYPE) static INLINE TYPE #else # define STATIC_INLINE_INTERRUPTS(TYPE) static TYPE @@ -261,7 +261,7 @@ # define EXTERN_DEVICE(TYPE) TYPE #endif -#if (DEVICE_INLINE & PSIM_INLINE_LOCALS) +#if (DEVICE_INLINE & INLINE_LOCALS) # define STATIC_INLINE_DEVICE(TYPE) static INLINE TYPE #else # define STATIC_INLINE_DEVICE(TYPE) static TYPE @@ -283,7 +283,7 @@ # define EXTERN_TREE(TYPE) TYPE #endif -#if (TREE_INLINE & PSIM_INLINE_LOCALS) +#if (TREE_INLINE & INLINE_LOCALS) # define STATIC_INLINE_TREE(TYPE) static INLINE TYPE #else # define STATIC_INLINE_TREE(TYPE) static TYPE @@ -305,7 +305,7 @@ # define EXTERN_SPREG(TYPE) TYPE #endif -#if (SPREG_INLINE & PSIM_INLINE_LOCALS) +#if (SPREG_INLINE & INLINE_LOCALS) # define STATIC_INLINE_SPREG(TYPE) static INLINE TYPE #else # define STATIC_INLINE_SPREG(TYPE) static TYPE @@ -328,7 +328,7 @@ #endif #if 0 /* this isn't used */ -#if (SEMANTICS_INLINE & PSIM_INLINE_LOCALS) +#if (SEMANTICS_INLINE & INLINE_LOCALS) # define STATIC_INLINE_SEMANTICS(TYPE) static INLINE TYPE #else # define STATIC_INLINE_SEMANTICS(TYPE) static TYPE @@ -352,7 +352,7 @@ #endif #if 0 /* this isn't used */ -#if (IDECODE_INLINE & PSIM_INLINE_LOCALS) +#if (IDECODE_INLINE & INLINE_LOCALS) # define STATIC_INLINE_IDECODE(TYPE) static INLINE TYPE #else # define STATIC_INLINE_IDECODE(TYPE) static TYPE @@ -376,7 +376,7 @@ #endif #if 0 /* this isn't used */ -#if (ICACHE_INLINE & PSIM_INLINE_LOCALS) +#if (ICACHE_INLINE & INLINE_LOCALS) # define STATIC_INLINE_ICACHE(TYPE) static INLINE TYPE #else # define STATIC_INLINE_ICACHE(TYPE) static TYPE @@ -400,7 +400,7 @@ #endif #if 0 /* this isn't used */ -#if (SUPPORT_INLINE & PSIM_INLINE_LOCALS) +#if (SUPPORT_INLINE & INLINE_LOCALS) # define STATIC_INLINE_SUPPORT(TYPE) static INLINE TYPE #else # define STATIC_INLINE_SUPPORT(TYPE) static TYPE @@ -423,7 +423,7 @@ # define EXTERN_OPTIONS(TYPE) TYPE #endif -#if (OPTIONS_INLINE & PSIM_INLINE_LOCALS) +#if (OPTIONS_INLINE & INLINE_LOCALS) # define STATIC_INLINE_OPTIONS(TYPE) static INLINE TYPE #else # define STATIC_INLINE_OPTIONS(TYPE) static TYPE @@ -445,7 +445,7 @@ # define EXTERN_OS_EMUL(TYPE) TYPE #endif -#if (OS_EMUL_INLINE & PSIM_INLINE_LOCALS) +#if (OS_EMUL_INLINE & INLINE_LOCALS) # define STATIC_INLINE_OS_EMUL(TYPE) static INLINE TYPE #else # define STATIC_INLINE_OS_EMUL(TYPE) static TYPE @@ -467,7 +467,7 @@ # define EXTERN_PSIM(TYPE) TYPE #endif -#if (PSIM_INLINE & PSIM_INLINE_LOCALS) +#if (PSIM_INLINE & INLINE_LOCALS) # define STATIC_INLINE_PSIM(TYPE) static INLINE TYPE #else # define STATIC_INLINE_PSIM(TYPE) static TYPE @@ -489,7 +489,7 @@ # define EXTERN_CAP(TYPE) TYPE #endif -#if (CAP_INLINE & PSIM_INLINE_LOCALS) +#if (CAP_INLINE & INLINE_LOCALS) # define STATIC_INLINE_CAP(TYPE) static INLINE TYPE #else # define STATIC_INLINE_CAP(TYPE) static TYPE diff --git a/sim/ppc/options.c b/sim/ppc/options.c index d20fb04e6ca..8aefcb16a6a 100644 --- a/sim/ppc/options.c +++ b/sim/ppc/options.c @@ -97,10 +97,10 @@ options_inline (int in) case /*1*/ REVEAL_MODULE: return "REVEAL_MODULE"; case /*2*/ INLINE_MODULE: return "INLINE_MODULE"; case /*3*/ REVEAL_MODULE|INLINE_MODULE: return "REVEAL_MODULE|INLINE_MODULE"; - case /*4*/ PSIM_INLINE_LOCALS: return "PSIM_LOCALS_INLINE"; - case /*5*/ PSIM_INLINE_LOCALS|REVEAL_MODULE: return "PSIM_INLINE_LOCALS|REVEAL_MODULE"; - case /*6*/ PSIM_INLINE_LOCALS|INLINE_MODULE: return "PSIM_INLINE_LOCALS|INLINE_MODULE"; - case /*7*/ ALL_INLINE: return "ALL_INLINE"; + case /*4*/ INLINE_LOCALS: return "INLINE_LOCALS"; + case /*5*/ INLINE_LOCALS|REVEAL_MODULE: return "INLINE_LOCALS|REVEAL_MODULE"; + case /*6*/ INLINE_LOCALS|INLINE_MODULE: return "INLINE_LOCALS|INLINE_MODULE"; + case /*7*/ ALL_C_INLINE: return "ALL_C_INLINE"; } return "0"; } diff --git a/sim/ppc/std-config.h b/sim/ppc/std-config.h index 7b8f9113859..75aaa588156 100644 --- a/sim/ppc/std-config.h +++ b/sim/ppc/std-config.h @@ -309,7 +309,7 @@ extern int current_stdio; the module is included into a file being compiled, calls to its funtions can be eliminated. 2 implies 1. - PSIM_INLINE_LOCALS: + INLINE_LOCALS: Make internal (static) functions within the module `inline'. @@ -317,7 +317,7 @@ extern int current_stdio; INCLUDE_MODULE == (REVEAL_MODULE | INLINE_MODULE) - ALL_INLINE == (REVEAL_MODULE | INLINE_MODULE | PSIM_INLINE_LOCALS) + ALL_C_INLINE == (REVEAL_MODULE | INLINE_MODULE | INLINE_LOCALS) In addition to this, modules have been put into two categories. @@ -415,11 +415,10 @@ extern int current_stdio; */ -#define REVEAL_MODULE 1 -#define INLINE_MODULE 2 +#include "../common/sim-inline.h" +#define REVEAL_MODULE H_REVEALS_MODULE +#define INLINE_MODULE C_REVEALS_MODULE #define INCLUDE_MODULE (INLINE_MODULE | REVEAL_MODULE) -#define PSIM_INLINE_LOCALS 4 -#define ALL_INLINE 7 /* Your compilers inline reserved word */ @@ -441,7 +440,7 @@ extern int current_stdio; /* Default macro to simplify control several of key the inlines */ #ifndef DEFAULT_INLINE -#define DEFAULT_INLINE PSIM_INLINE_LOCALS +#define DEFAULT_INLINE INLINE_LOCALS #endif /* Code that converts between hosts and target byte order. Used on @@ -450,21 +449,21 @@ extern int current_stdio; can inline for all callers */ #ifndef SIM_ENDIAN_INLINE -#define SIM_ENDIAN_INLINE (DEFAULT_INLINE ? ALL_INLINE : 0) +#define SIM_ENDIAN_INLINE (DEFAULT_INLINE ? ALL_C_INLINE : 0) #endif /* Low level bit manipulation routines. This module can inline for all callers */ #ifndef BITS_INLINE -#define BITS_INLINE (DEFAULT_INLINE ? ALL_INLINE : 0) +#define BITS_INLINE (DEFAULT_INLINE ? ALL_C_INLINE : 0) #endif /* Code that gives access to various CPU internals such as registers. Used every time an instruction is executed */ #ifndef CPU_INLINE -#define CPU_INLINE (DEFAULT_INLINE ? ALL_INLINE : 0) +#define CPU_INLINE (DEFAULT_INLINE ? ALL_C_INLINE : 0) #endif /* Code that translates between an effective and real address. Used @@ -485,14 +484,14 @@ extern int current_stdio; Called once per instruction cycle */ #ifndef EVENTS_INLINE -#define EVENTS_INLINE (DEFAULT_INLINE ? ALL_INLINE : 0) +#define EVENTS_INLINE (DEFAULT_INLINE ? ALL_C_INLINE : 0) #endif /* Code monotoring the processors performance. It counts events on every instruction cycle */ #ifndef MON_INLINE -#define MON_INLINE (DEFAULT_INLINE ? ALL_INLINE : 0) +#define MON_INLINE (DEFAULT_INLINE ? ALL_C_INLINE : 0) #endif /* Code called on the rare occasions that an interrupt occures. */ @@ -515,7 +514,7 @@ extern int current_stdio; a jump table. */ #ifndef DEVICE_INLINE -#define DEVICE_INLINE (DEFAULT_INLINE ? PSIM_INLINE_LOCALS : 0) +#define DEVICE_INLINE (DEFAULT_INLINE ? INLINE_LOCALS : 0) #endif /* Code called used while the device tree is being built. @@ -523,7 +522,7 @@ extern int current_stdio; Inlining this is of no benefit */ #ifndef TREE_INLINE -#define TREE_INLINE (DEFAULT_INLINE ? PSIM_INLINE_LOCALS : 0) +#define TREE_INLINE (DEFAULT_INLINE ? INLINE_LOCALS : 0) #endif /* Code called whenever information on a Special Purpose Register is @@ -563,7 +562,7 @@ extern int current_stdio; code is reduced. */ #ifndef SUPPORT_INLINE -#define SUPPORT_INLINE PSIM_INLINE_LOCALS +#define SUPPORT_INLINE INLINE_LOCALS #endif /* Model specific code used in simulating functional units. Note, it actaully @@ -588,13 +587,13 @@ extern int current_stdio; into this file */ #ifndef IDECOCE_INLINE -#define IDECODE_INLINE PSIM_INLINE_LOCALS +#define IDECODE_INLINE INLINE_LOCALS #endif /* psim, isn't actually inlined */ #ifndef PSIM_INLINE -#define PSIM_INLINE PSIM_INLINE_LOCALS +#define PSIM_INLINE INLINE_LOCALS #endif /* Code to emulate os or rom compatibility. This code is called via a -- 2.30.2