|
From: | Philippe Mathieu-Daudé |
Subject: | Re: [PATCH v6 5/9] target/riscv: remove 'cpu_vl' global |
Date: | Thu, 22 Feb 2024 08:59:13 +0100 |
User-agent: | Mozilla Thunderbird |
On 21/2/24 22:31, Daniel Henrique Barboza wrote:
At this moment the global is used only in do_vsetvl(). Do a direct env load in do_vsetvl() to read 'vl' and remove the global. Suggested-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com> --- target/riscv/insn_trans/trans_rvv.c.inc | 2 +- target/riscv/translate.c | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-)
I was expecting this to be squashed in previous patch, but it is indeed clearer as a separate one. Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
[Prev in Thread] | Current Thread | [Next in Thread] |