qemu-riscv
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 10/11] RISC-V: Adding T-Head FMemIdx extension


From: Richard Henderson
Subject: Re: [PATCH 10/11] RISC-V: Adding T-Head FMemIdx extension
Date: Thu, 8 Sep 2022 08:45:08 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0

On 9/6/22 13:22, Christoph Muellner wrote:
@@ -732,6 +733,7 @@ static int ex_rvc_shifti(DisasContext *ctx, int imm)
  #include "decode-xtheadbs.c.inc"
  #include "decode-xtheadcmo.c.inc"
  #include "decode-xtheadcondmov.c.inc"
+#include "decode-xtheadfmemidx.c.inc"
  #include "decode-xtheadmac.c.inc"
  #include "decode-xtheadmemidx.c.inc"
  #include "decode-xtheadmempair.c.inc"
@@ -1061,6 +1063,7 @@ static void decode_opc(CPURISCVState *env, DisasContext 
*ctx, uint16_t opcode)
          { has_xtheadbs_p, decode_xtheadbs },
          { has_xtheadcmo_p, decode_xtheadcmo },
          { has_xtheadcondmov_p, decode_xtheadcondmov },
+        { has_xtheadfmemidx_p, decode_xtheadfmemidx },
          { has_xtheadmac_p, decode_xtheadmac },
          { has_xtheadmemidx_p, decode_xtheadmemidx },
          { has_xtheadmempair_p, decode_xtheadmempair },

I think you should have a single decoder for all of the xthread extensions, and each translate function should test for the individual extension. You know up-front that these extensions do not conflict.


r~



reply via email to

[Prev in Thread] Current Thread [Next in Thread]