Re: [Qemu-devel] [PATCH v6 39/77] target/mips: Add emulation of DSP ASE for nanoMIPS - part 2

[Aleksandar Markovic]

> +                case NM_BPOSGE32C:
> +                    check_dsp(ctx);
> +                    {
> +                        int32_t imm = extract32(ctx->opcode, 1, 13) |
> +                                      extract32(ctx->opcode, 0, 1) << 13;
> +
> +                        gen_compute_branch(ctx, OPC_BPOSGE32, 4, -1, -2,
> +                                           imm, 4);
> +                    }
> +                    break;


BPOSGE32C is introduced in DSP-R3. Shouldn't there be check_dspr3(), and 
applied here?

>From the code, it turns out that the only difference between BPOSGE32 
>(supported in previous mips DSP specifications) and BPOSGE32C (nanoMIPS DSP 
>supported) is offset calculation. Is this really the only difference - from 
>the documentation?

Aleksandar M.