Commit 1d023a52 authored by Peter Maydell's avatar Peter Maydell
Browse files

Merge remote-tracking branch 'remotes/palmer/tags/riscv-for-4.0-rc0' into staging 



A Single RISC-V Patch for 4.0-rc0

There was a regression introduced by the decodetree conversion that has
a fairly straight-forward fix.  Since this fixes bugs that everyone has
hit I'd like to target it for rc0.

# gpg: Signature made Mon 18 Mar 2019 05:26:07 GMT
# gpg:                using RSA key 00CE76D1834960DFCE886DF8EF4CA1502CCBAB41
# gpg:                issuer "palmer@dabbelt.com"
# gpg: Good signature from "Palmer Dabbelt <palmer@dabbelt.com>" [unknown]
# gpg:                 aka "Palmer Dabbelt <palmer@sifive.com>" [unknown]
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: 00CE 76D1 8349 60DF CE88  6DF8 EF4C A150 2CCB AB41

* remotes/palmer/tags/riscv-for-4.0-rc0:
  target/riscv: Fix manually parsed 16 bit insn

Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
parents d4e65539 f330433b

target/riscv/insn_trans/trans_rvc.inc.c

+25 −5
Original line number Diff line number Diff line
@@ -44,10 +44,19 @@ static bool trans_c_flw_ld(DisasContext *ctx, arg_c_flw_ld *a) 
{

#ifdef TARGET_RISCV32

    /* C.FLW ( RV32FC-only ) */

    return false;

    REQUIRE_FPU;

    REQUIRE_EXT(ctx, RVF);



    arg_c_lw tmp;

    decode_insn16_extract_cl_w(&tmp, ctx->opcode);

    arg_flw arg = { .rd = tmp.rd, .rs1 = tmp.rs1, .imm = tmp.uimm };

    return trans_flw(ctx, &arg);

#else

    /* C.LD ( RV64C/RV128C-only ) */

    return false;

    arg_c_fld tmp;

    decode_insn16_extract_cl_d(&tmp, ctx->opcode);

    arg_ld arg = { .rd = tmp.rd, .rs1 = tmp.rs1, .imm = tmp.uimm };

    return trans_ld(ctx, &arg);

#endif

}
@@ -67,10 +76,19 @@ static bool trans_c_fsw_sd(DisasContext *ctx, arg_c_fsw_sd *a) 
{

#ifdef TARGET_RISCV32

    /* C.FSW ( RV32FC-only ) */

    return false;

    REQUIRE_FPU;

    REQUIRE_EXT(ctx, RVF);



    arg_c_sw tmp;

    decode_insn16_extract_cs_w(&tmp, ctx->opcode);

    arg_fsw arg = { .rs1 = tmp.rs1, .rs2 = tmp.rs2, .imm = tmp.uimm };

    return trans_fsw(ctx, &arg);

#else

    /* C.SD ( RV64C/RV128C-only ) */

    return false;

    arg_c_fsd tmp;

    decode_insn16_extract_cs_d(&tmp, ctx->opcode);

    arg_sd arg = { .rs1 = tmp.rs1, .rs2 = tmp.rs2, .imm = tmp.uimm };

    return trans_sd(ctx, &arg);

#endif

}
@@ -88,7 +106,9 @@ static bool trans_c_jal_addiw(DisasContext *ctx, arg_c_jal_addiw *a) 
{

#ifdef TARGET_RISCV32

    /* C.JAL */

    arg_jal arg = { .rd = 1, .imm = a->imm };

    arg_c_j tmp;

    decode_insn16_extract_cj(&tmp, ctx->opcode);

    arg_jal arg = { .rd = 1, .imm = tmp.imm };

    return trans_jal(ctx, &arg);

#else

    /* C.ADDIW */