Reviewed-by: Michael Rolnik <[email protected]> On Sat, Nov 19, 2022 at 7:56 AM Pavel Dovgalyuk <[email protected]> wrote:
> Bit vector for features has 64 bits. This patch fixes bit shifts in > avr_feature and set_avr_feature functions to be 64-bit too. > > Signed-off-by: Pavel Dovgalyuk <[email protected]> > --- > target/avr/cpu.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/target/avr/cpu.h b/target/avr/cpu.h > index cfdc0ecb70..8295e50fa0 100644 > --- a/target/avr/cpu.h > +++ b/target/avr/cpu.h > @@ -166,12 +166,12 @@ vaddr avr_cpu_gdb_adjust_breakpoint(CPUState *cpu, > vaddr addr); > > static inline int avr_feature(CPUAVRState *env, AVRFeature feature) > { > - return (env->features & (1U << feature)) != 0; > + return (env->features & (1ULL << feature)) != 0; > } > > static inline void set_avr_feature(CPUAVRState *env, int feature) > { > - env->features |= (1U << feature); > + env->features |= (1ULL << feature); > } > > #define cpu_list avr_cpu_list > > -- Best Regards, Michael Rolnik
