On Wed, 27 Aug 2025 04:04, Richard Henderson <richard.hender...@linaro.org>
wrote:
>Reduce the places that know about field types by 1.
>
>Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
>---
Reviewed-by: Manos Pitsidianakis <manos.pitsidiana...@linaro.org>
> target/arm/cpu.c | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
>
>diff --git a/target/arm/cpu.c b/target/arm/cpu.c
>index e2b2337399..ed40e102fc 100644
>--- a/target/arm/cpu.c
>+++ b/target/arm/cpu.c
>@@ -192,14 +192,8 @@ static void cp_reg_reset(gpointer key, gpointer value,
>gpointer opaque)
> * This is basically only used for fields in non-core coprocessors
> * (like the pxa2xx ones).
> */
>- if (!ri->fieldoffset) {
>- return;
>- }
>-
>- if (cpreg_field_is_64bit(ri)) {
>- CPREG_FIELD64(&cpu->env, ri) = ri->resetvalue;
>- } else {
>- CPREG_FIELD32(&cpu->env, ri) = ri->resetvalue;
>+ if (ri->fieldoffset) {
>+ raw_write(&cpu->env, ri, ri->resetvalue);
> }
> }
>
>--
>2.43.0
>
>
