Re: [PATCH] target/i386: generate simpler code for ROL/ROR with immediate count

2024-05-27 Thread Philippe Mathieu-Daudé
On 22/5/24 14:39, Paolo Bonzini wrote: gen_rot_carry and gen_rot_overflow are meant to be called with count == NULL if the count cannot be zero. However this is not done in gen_ROL and gen_ROR, and writing everywhere "can_be_zero ? count : NULL" is burdensome and less readable. Just pass can_be

Re: [PATCH] target/i386: generate simpler code for ROL/ROR with immediate count

2024-05-22 Thread Richard Henderson
On 5/22/24 05:39, Paolo Bonzini wrote: gen_rot_carry and gen_rot_overflow are meant to be called with count == NULL if the count cannot be zero. However this is not done in gen_ROL and gen_ROR, and writing everywhere "can_be_zero ? count : NULL" is burdensome and less readable. Just pass can_be

[PATCH] target/i386: generate simpler code for ROL/ROR with immediate count

2024-05-22 Thread Paolo Bonzini
gen_rot_carry and gen_rot_overflow are meant to be called with count == NULL if the count cannot be zero. However this is not done in gen_ROL and gen_ROR, and writing everywhere "can_be_zero ? count : NULL" is burdensome and less readable. Just pass can_be_zero as a separate argument. gen_RCL an