target/arm: Remove dead code relating to SABA and UABA
In commitcfdb2c0c95
("target/arm: Vectorize SABA/UABA") we replaced the old handling of SABA/UABA with a vectorized implementation which returns early rather than falling into the loop-ever-elements code. We forgot to delete the part of the old looping code that did the accumulate step, and Coverity correctly warns (CID 1428955) that this code is now dead. Delete it. Fixes:cfdb2c0c95
Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-id: 20200619171547.29780-1-peter.maydell@linaro.org
This commit is contained in:
parent
55c812b742
commit
ced7e8edb2
|
@ -11370,18 +11370,6 @@ static void disas_simd_3same_int(DisasContext *s, uint32_t insn)
|
|||
genfn(tcg_res, tcg_op1, tcg_op2);
|
||||
}
|
||||
|
||||
if (opcode == 0xf) {
|
||||
/* SABA, UABA: accumulating ops */
|
||||
static NeonGenTwoOpFn * const fns[3] = {
|
||||
gen_helper_neon_add_u8,
|
||||
gen_helper_neon_add_u16,
|
||||
tcg_gen_add_i32,
|
||||
};
|
||||
|
||||
read_vec_element_i32(s, tcg_op1, rd, pass, MO_32);
|
||||
fns[size](tcg_res, tcg_op1, tcg_res);
|
||||
}
|
||||
|
||||
write_vec_element_i32(s, tcg_res, rd, pass, MO_32);
|
||||
|
||||
tcg_temp_free_i32(tcg_res);
|
||||
|
|
Loading…
Reference in a new issue