On 12 September 2017 at 17:25, Richard Henderson <richard.hender...@linaro.org> wrote: > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > --- > target/arm/cpu.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/arm/cpu.h b/target/arm/cpu.h > index 98b9b26fd3..419f008277 100644 > --- a/target/arm/cpu.h > +++ b/target/arm/cpu.h > @@ -486,7 +486,7 @@ typedef struct CPUARMState { > * the two execution states, and means we do not need to explicitly > * map these registers when changing states. > */ > - float64 regs[64]; > + float64 regs[64] __attribute__((aligned(16))); > > uint32_t xregs[16]; > /* We store these fpcsr fields separately for convenience. */ > -- > 2.13.5
I notice we have a QEMU_ALIGNED() macro to wrap the __attribute__, though we use it less often than not at the moment... thanks -- PMM