Re: [PATCH 15/15] Hexagon (target/hexagon) CABAC decode bin

2021-03-25 Thread Richard Henderson
On 3/24/21 8:50 PM, Taylor Simpson wrote: +#define fINSERT_RANGE(INREG, HIBIT, LOWBIT, INVAL) \ +INREG = (((HIBIT) - (LOWBIT) - 1) ? \ +deposit64((INREG), (LOWBIT), ((HIBIT) - (LOWBIT) + 1), (INVAL)) : \ +INREG) The -1 and +1 on the length of the field doesn't look right.

[PATCH 15/15] Hexagon (target/hexagon) CABAC decode bin

2021-03-24 Thread Taylor Simpson
The following instruction is added S2_cabacdecbinRdd32=decbin(Rss32,Rtt32) Test cases added to tests/tcg/hexagon/misc.c Signed-off-by: Taylor Simpson --- target/hexagon/arch.c | 91 +++ target/hexagon/arch.h | 4