On 10/12/18 10:30 AM, Bastian Koppelmann wrote: > +#define EX_SH(amount) \ > + static int64_t ex_shift_##amount(int imm) \ > + { \ > + return imm << amount; \ > + }
The int64_t return doesn't help, because it'll be stored back into a struct that uses just "int". If you have a shift that goes outside that, you'd have to handle it in the trans_* function. Which is not the case here, at least. Otherwise, Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~