On 07/19/2018 05:54 AM, Stefan Markovic wrote: > +static void gen_pool32axf_nanomips_insn(CPUMIPSState *env, DisasContext *ctx) > +{ > + int rt = (ctx->opcode >> 21) & 0x1f; > + int rs = (ctx->opcode >> 16) & 0x1f; > + > + switch ((ctx->opcode >> 6) & 0x07) {
extract32. > case NM_POOL32A7: > + { > + switch ((ctx->opcode >> 3) & 0x07) { > + case NM_POOL32AXF: > + gen_pool32axf_nanomips_insn(env, ctx); > + break; > + } > + } Bad indentation of a block that need not exist anyway. Otherwise, Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~