Re: [PATCH V14 2/8] target/mips: Add loongson-ext lswc2 group of instructions (Part 1)

[Philippe Mathieu-Daudé]

On 10/16/20 8:51 AM, Huacai Chen wrote:
> From: Jiaxun Yang <jiaxun.yang@flygoat.com>
>
> LWC2 & SWC2 have been rewritten by Loongson EXT vendor ASE
> as "load/store quad word" and "shifted load/store" groups of
> instructions.
>
> This patch add implementation of these instructions:
> gslq: load 16 bytes to GPR
> gssq: store 16 bytes from GPR
> gslqc1: load 16 bytes to FPR
> gssqc1: store 16 bytes from FPR
>
> Details of Loongson-EXT is here:
> https://github.com/FlyGoat/loongson-insn/blob/master/loongson-ext.md
>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> Signed-off-by: Huacai Chen <chenhc@lemote.com>
> ---
>   target/mips/translate.c | 86 +++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 86 insertions(+)

Build failure:

target/mips/translate.c: In function ‘gen_loongson_lswc2’:
target/mips/translate.c:5928:14: error: unused variable ‘fp0’ 
[-Werror=unused-variable]
 5928 |     TCGv_i32 fp0;
      |              ^~~
target/mips/translate.c:5927:18: error: unused variable ‘t2’ 
[-Werror=unused-variable]
 5927 |     TCGv t0, t1, t2;
      |                  ^~
target/mips/translate.c:5927:14: error: unused variable ‘t1’ 
[-Werror=unused-variable]
 5927 |     TCGv t0, t1, t2;
      |              ^~