From 46af765bbb30610fd02347f010fce6ceea12aba5 Mon Sep 17 00:00:00 2001 From: Chen Qun Date: Sat, 27 Feb 2021 16:45:23 -0500 Subject: [PATCH] target/arm/translate-a64:Remove redundant statement in disas_simd_two_reg_misc_fp16() Clang static code analyzer show warning: target/arm/translate-a64.c:13007:5: warning: Value stored to 'rd' is never read rd = extract32(insn, 0, 5); ^ ~~~~~~~~~~~~~~~~~~~~~ target/arm/translate-a64.c:13008:5: warning: Value stored to 'rn' is never read rn = extract32(insn, 5, 5); ^ ~~~~~~~~~~~~~~~~~~~~~ Backports fa71dd531c12ad9a05cdd78392e9fc2a30ea921d --- qemu/target/arm/translate-a64.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/qemu/target/arm/translate-a64.c b/qemu/target/arm/translate-a64.c index aeaa85a2..d54a78ac 100644 --- a/qemu/target/arm/translate-a64.c +++ b/qemu/target/arm/translate-a64.c @@ -13324,9 +13324,6 @@ static void disas_simd_two_reg_misc_fp16(DisasContext *s, uint32_t insn) fpop = deposit32(opcode, 5, 1, a); fpop = deposit32(fpop, 6, 1, u); - rd = extract32(insn, 0, 5); - rn = extract32(insn, 5, 5); - switch (fpop) { case 0x1d: /* SCVTF */ case 0x5d: /* UCVTF */