Hello, On Fri, Aug 17, 2018 at 6:04 PM Peter Maydell <peter.mayd...@linaro.org> wrote: > > On 9 August 2018 at 05:21, Richard Henderson > <richard.hender...@linaro.org> wrote: > > Saves about 8k code size in qemu-aarch64. > > > > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > > --- > > target/arm/cpu.h | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/target/arm/cpu.h b/target/arm/cpu.h > > index aedaf2631e..ed51a2f5aa 100644 > > --- a/target/arm/cpu.h > > +++ b/target/arm/cpu.h > > @@ -918,7 +918,15 @@ void aarch64_sync_64_to_32(CPUARMState *env); > > > > static inline bool is_a64(CPUARMState *env) > > { > > +#ifdef CONFIG_USER_ONLY > > +# ifdef TARGET_AARCH64 > > + return true; > > +# else > > + return false; > > +# endif > > +#else > > return env->aarch64; > > +#endif > > } > > And again. I don't want to pepper the code with ifdefs if > we can do the right thing without them.
FWIW I find it more readable with the ifdef's (here and in the previous patches) and I guess that helps the compiler too. Thanks, Laurent