Re: [PATCH v2 06/13] target/riscv: Use REQUIRE_64BIT in amo_check64

[LIU Zhiwei]

On 2021/10/14 上午4:50, Richard Henderson wrote:

Use the same REQUIRE_64BIT check that we use elsewhere,
rather than open-coding the use of is_32bit.

Signed-off-by: Richard Henderson <richard.hender...@linaro.org>

Reviewed-by: LIU Zhiwei<zhiwei_...@c-sky.com>

Zhiwei

---
  target/riscv/insn_trans/trans_rvv.c.inc | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/target/riscv/insn_trans/trans_rvv.c.inc 
b/target/riscv/insn_trans/trans_rvv.c.inc
index fa451938f1..bbc5c93ef1 100644
--- a/target/riscv/insn_trans/trans_rvv.c.inc
+++ b/target/riscv/insn_trans/trans_rvv.c.inc
@@ -743,7 +743,8 @@ static bool amo_check(DisasContext *s, arg_rwdvm* a)

static bool amo_check64(DisasContext *s, arg_rwdvm* a)

  {
-    return !is_32bit(s) && amo_check(s, a);
+    REQUIRE_64BIT(s);
+    return amo_check(s, a);
  }

GEN_VEXT_TRANS(vamoswapw_v, 0, rwdvm, amo_op, amo_check)