target/arm: Use neon_element_offset in vfp_reg_offset
This seems a bit more readable than using offsetof CPU_DoubleU. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Message-id: 20201030022618.785675-5-richard.henderson@linaro.org Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
parent
0f2cdc8227
commit
d8719785fd
@ -1122,18 +1122,13 @@ static long neon_element_offset(int reg, int element, MemOp size)
|
|||||||
return neon_full_reg_offset(reg) + ofs;
|
return neon_full_reg_offset(reg) + ofs;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline long vfp_reg_offset(bool dp, unsigned reg)
|
/* Return the offset of a VFP Dreg (dp = true) or VFP Sreg (dp = false). */
|
||||||
|
static long vfp_reg_offset(bool dp, unsigned reg)
|
||||||
{
|
{
|
||||||
if (dp) {
|
if (dp) {
|
||||||
return offsetof(CPUARMState, vfp.zregs[reg >> 1].d[reg & 1]);
|
return neon_element_offset(reg, 0, MO_64);
|
||||||
} else {
|
} else {
|
||||||
long ofs = offsetof(CPUARMState, vfp.zregs[reg >> 2].d[(reg >> 1) & 1]);
|
return neon_element_offset(reg >> 1, reg & 1, MO_32);
|
||||||
if (reg & 1) {
|
|
||||||
ofs += offsetof(CPU_DoubleU, l.upper);
|
|
||||||
} else {
|
|
||||||
ofs += offsetof(CPU_DoubleU, l.lower);
|
|
||||||
}
|
|
||||||
return ofs;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user