On 12/29/18 12:52 AM, Mark Cave-Ayland wrote: > Richard points out that these macros suffer from a -fsanitize=shift bug in > that > they improperly handle n == 0 turning it into a shift by 32/64 respectively. > Replace them with QEMU's existing ror32() and ror64() functions instead. > > Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> > --- > target/ppc/int_helper.c | 48 ++++++++++++++++++++---------------------------- > 1 file changed, 20 insertions(+), 28 deletions(-)
Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~