On 25/2/23 11:09, Richard Henderson wrote:
On 2/25/23 00:05, Philippe Mathieu-Daudé wrote:
On 25/2/23 09:59, Richard Henderson wrote:
Since tcg_temp_new_* is now identical, use those.
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
---
target/arm/translate-sve.c | 6 +++---
target/arm/translate.c | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/target/arm/translate-sve.c b/target/arm/translate-sve.c
index 02150d93e8..718a5bce1b 100644
--- a/target/arm/translate-sve.c
+++ b/target/arm/translate-sve.c
@@ -4342,7 +4342,7 @@ void gen_sve_ldr(DisasContext *s, TCGv_ptr
base, int vofs,
tcg_temp_free_i64(t0);
} else {
TCGLabel *loop = gen_new_label();
- TCGv_ptr tp, i = tcg_const_local_ptr(0);
+ TCGv_ptr tp, i = tcg_const_ptr(0);
Can we directly switch to tcg_constant_ptr(NULL)?
No, i is written. This ought to get changed to tcg_temp_new +
tcg_gen_movi, but that'll wait for a different patch set.
Indeed. Sure :)