From 2ac01a4efceacb9f2f9433db636545885296da0a Mon Sep 17 00:00:00 2001 From: Wilco Dijkstra <wilco.dijkstra@arm.com> Date: Thu, 17 Oct 2024 14:33:44 +0000 Subject: [PATCH] AArch64: Remove redundant check in aarch64_simd_mov The split condition in aarch64_simd_mov uses aarch64_simd_special_constant_p. While doing the split, it checks the mode before calling aarch64_maybe_generate_simd_constant. This risky since it may result in unexpectedly calling aarch64_split_simd_move instead of aarch64_maybe_generate_simd_constant. Since the mode is already checked, remove the spurious explicit mode check. gcc/ChangeLog: * config/aarch64/aarch64-simd.md (aarch64_simd_mov<VQMOV:mode>): Remove redundant mode check. --- gcc/config/aarch64/aarch64-simd.md | 1 - 1 file changed, 1 deletion(-) diff --git a/gcc/config/aarch64/aarch64-simd.md b/gcc/config/aarch64/aarch64-simd.md index 8826f9d68885..04851524fdea 100644 --- a/gcc/config/aarch64/aarch64-simd.md +++ b/gcc/config/aarch64/aarch64-simd.md @@ -208,7 +208,6 @@ else { if (FP_REGNUM_P (REGNO (operands[0])) - && <MODE>mode == V2DImode && aarch64_maybe_generate_simd_constant (operands[0], operands[1], <MODE>mode)) ; -- GitLab