| Message ID | 20241021151253.36443-1-philmd@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | target/mips: Remove unused CPUMIPSState::current_fpu field | expand |
On 10/21/24 08:12, Philippe Mathieu-Daudé wrote: > The 'current_fpu' field is unused since commit f01be154589 > ("Move the active FPU registers into env again, and use more > TCG registers to access them"). > > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> > --- > target/mips/cpu.h | 1 - > target/mips/sysemu/machine.c | 2 +- > 2 files changed, 1 insertion(+), 2 deletions(-) > > diff --git a/target/mips/cpu.h b/target/mips/cpu.h > index 6a4c4ea683a..3c235f39ffc 100644 > --- a/target/mips/cpu.h > +++ b/target/mips/cpu.h > @@ -530,7 +530,6 @@ typedef struct CPUArchState { > CPUMIPSFPUContext active_fpu; > > uint32_t current_tc; > - uint32_t current_fpu; > > uint32_t SEGBITS; > uint32_t PABITS; > diff --git a/target/mips/sysemu/machine.c b/target/mips/sysemu/machine.c > index 91cd9f19002..8af11fd896b 100644 > --- a/target/mips/sysemu/machine.c > +++ b/target/mips/sysemu/machine.c > @@ -242,7 +242,7 @@ const VMStateDescription vmstate_mips_cpu = { > > /* CPU metastate */ > VMSTATE_UINT32(env.current_tc, MIPSCPU), > - VMSTATE_UINT32(env.current_fpu, MIPSCPU), > + VMSTATE_UNUSED(sizeof(uint32_t)), /* was current_fpu */ > VMSTATE_INT32(env.error_code, MIPSCPU), > VMSTATE_UINTTL(env.btarget, MIPSCPU), > VMSTATE_UINTTL(env.bcond, MIPSCPU), Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
On 10/21/24 08:12, Philippe Mathieu-Daudé wrote: > The 'current_fpu' field is unused since commit f01be154589 > ("Move the active FPU registers into env again, and use more > TCG registers to access them"). > > Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org> > --- > target/mips/cpu.h | 1 - > target/mips/sysemu/machine.c | 2 +- > 2 files changed, 1 insertion(+), 2 deletions(-) Reviewed-by: Richard Henderson <richard.henderson@linaro.org> r~
diff --git a/target/mips/cpu.h b/target/mips/cpu.h index 6a4c4ea683a..3c235f39ffc 100644 --- a/target/mips/cpu.h +++ b/target/mips/cpu.h @@ -530,7 +530,6 @@ typedef struct CPUArchState { CPUMIPSFPUContext active_fpu; uint32_t current_tc; - uint32_t current_fpu; uint32_t SEGBITS; uint32_t PABITS; diff --git a/target/mips/sysemu/machine.c b/target/mips/sysemu/machine.c index 91cd9f19002..8af11fd896b 100644 --- a/target/mips/sysemu/machine.c +++ b/target/mips/sysemu/machine.c @@ -242,7 +242,7 @@ const VMStateDescription vmstate_mips_cpu = { /* CPU metastate */ VMSTATE_UINT32(env.current_tc, MIPSCPU), - VMSTATE_UINT32(env.current_fpu, MIPSCPU), + VMSTATE_UNUSED(sizeof(uint32_t)), /* was current_fpu */ VMSTATE_INT32(env.error_code, MIPSCPU), VMSTATE_UINTTL(env.btarget, MIPSCPU), VMSTATE_UINTTL(env.bcond, MIPSCPU),
The 'current_fpu' field is unused since commit f01be154589 ("Move the active FPU registers into env again, and use more TCG registers to access them"). Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> --- target/mips/cpu.h | 1 - target/mips/sysemu/machine.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-)