On 08/16/2018 07:57 AM, Aleksandar Markovic wrote:
> +    switch (opc) {
> +    case NM_POOL32AXF_2_0_7:
> +        switch (extract32(ctx->opcode, 9, 3)) {

For future cleanup, I can't help but thinking you'd be better served by
extracting a different set of opcode bits and having a single switch.

That said, it all appears to all be correct, so
Reviewed-by: Richard Henderson <richard.hender...@linaro.org>


r~

Reply via email to