On Fri, Jul 9, 2021 at 2:48 PM Richard Henderson <richard.hender...@linaro.org> wrote: > > This function is now unused. > The corresponding gen_set_gpr function is still in use. > > Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > --- > target/riscv/translate.c | 17 +++++------------ > 1 file changed, 5 insertions(+), 12 deletions(-) > > diff --git a/target/riscv/translate.c b/target/riscv/translate.c > index 8ff75a5798..a6c850e9d3 100644 > --- a/target/riscv/translate.c > +++ b/target/riscv/translate.c > @@ -195,15 +195,6 @@ static void gen_goto_tb(DisasContext *ctx, int n, > target_ulong dest) > /* Wrapper for getting reg values - need to check of reg is zero since > * cpu_gpr[0] is not actually allocated > */ > -static inline void gen_get_gpr(TCGv t, int reg_num) > -{ > - if (reg_num == 0) { > - tcg_gen_movi_tl(t, 0); > - } else { > - tcg_gen_mov_tl(t, cpu_gpr[reg_num]); > - } > -} > - > static TCGv gpr_src(DisasContext *ctx, int reg_num) > { > if (reg_num == 0) { > @@ -1046,9 +1037,11 @@ void riscv_translate_init(void) > { > int i; > > - /* cpu_gpr[0] is a placeholder for the zero register. Do not use it. */ > - /* Use the gen_set_gpr and gen_get_gpr helper functions when accessing */ > - /* registers, unless you specifically block reads/writes to reg 0 */ > + /* > + * cpu_gpr[0] is a placeholder for the zero register. Do not use it. > + * Use the gpr_src and gpr_dst helper functions when accessing regs, > + * unless you specifically block reads/writes to reg 0. > + */ > cpu_gpr[0] = NULL; > > for (i = 1; i < 32; i++) { > -- > 2.25.1 > >