[Top][All Lists]

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

Re: [PATCH v2 09/13] target/riscv: Replace DisasContext.w with DisasCont

From: Richard Henderson
Subject: Re: [PATCH v2 09/13] target/riscv: Replace DisasContext.w with DisasContext.ol
Date: Thu, 14 Oct 2021 08:39:32 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0

On 10/14/21 1:57 AM, Frédéric Pétrot wrote:
diff --git a/target/riscv/translate.c b/target/riscv/translate.c
index 5724a62bb0..6ab5c6aa58 100644
--- a/target/riscv/translate.c
+++ b/target/riscv/translate.c
@@ -67,7 +67,7 @@ typedef struct DisasContext {
          to any system register, which includes CSR_FRM, so we do not have
          to reset this known value.  */
       int frm;
-    bool w;
+    RISCVMXL ol;

Why not directly use the xl?

   Hi Zhiwei,

   I am not speaking for Richard, but my understanding is that 'ol' is linked to
   the instruction being translated, suffixed by 'w' in rv64 and 'w' and 'd' in
   rv128, while 'xl' is the value in mstatus (or misa) depending on the register
   size of the current execution (mxl, sxl, uxl).



reply via email to

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