On 11/27/2016 11:53 AM, Laurent Vivier wrote:
>>              tcg_gen_andi_i32(tmp, tmp, 31);
>> >              mask = tcg_const_i32(0x7fffffffu);
>> >              tcg_gen_shr_i32(mask, mask, tmp);
>> > +            if (!TCGV_IS_UNUSED(tlen)) {
>> > +                tcg_gen_mov_i32(tlen, tmp);
> We must add 1 here, otherwise the value stays between 0 and 31, not 1
> and 32:
> 
> +                tcg_gen_addi_i32(tlen, tmp, 1);
> 

Yep, good catch.


r~

Reply via email to