https://gcc.gnu.org/g:17cce5f59a4d9cdb87f59c6841da89c9ea2ef8d5

commit r14-11633-g17cce5f59a4d9cdb87f59c6841da89c9ea2ef8d5
Author: Richard Sandiford <richard.sandif...@arm.com>
Date:   Wed Apr 16 13:20:27 2025 +0100

    aarch64: Use force_lowpart_subreg in a BFI splitter [PR119133]
    
    lowpart_subreg ICEs are the gift that keeps giving.  This is another
    case where we need to use force_lowpart_subreg instead, to handle
    cases where the input is already a subreg and where the combined
    subreg is not allowed as a single operation.
    
    We don't need to check can_create_pseudo_p since the input should
    be a hard register rather than a subreg if !can_create_pseudo_p.
    
    gcc/
            PR target/119133
            * config/aarch64/aarch64.md
            (*aarch64_bfi<GPI:mode><ALLX:mode>_<SUBDI_BITS>): Use
            force_lowpart_subreg.
    
    gcc/testsuite/
            PR target/119133
            * gcc.dg/torture/pr119133.c: New test.
    
    (cherry picked from commit 5ae621e2e86c00d1fb13ef6839d0c3bace762ac8)

Diff:
---
 gcc/config/aarch64/aarch64.md           |  4 ++--
 gcc/testsuite/gcc.dg/torture/pr119133.c | 10 ++++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
index d3c381e82ce4..1be8322c4ee0 100644
--- a/gcc/config/aarch64/aarch64.md
+++ b/gcc/config/aarch64/aarch64.md
@@ -6161,8 +6161,8 @@
                          (match_dup 1))
        (match_dup 2))]
   {
-    operands[2] = lowpart_subreg (<GPI:MODE>mode, operands[2],
-                                 <ALLX:MODE>mode);
+    operands[2] = force_lowpart_subreg (<GPI:MODE>mode, operands[2],
+                                       <ALLX:MODE>mode);
   }
   [(set_attr "type" "bfm,neon_ins_q,neon_ins_q")
    (set_attr "arch" "*,simd,simd")]
diff --git a/gcc/testsuite/gcc.dg/torture/pr119133.c 
b/gcc/testsuite/gcc.dg/torture/pr119133.c
new file mode 100644
index 000000000000..f0c8f734c866
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/torture/pr119133.c
@@ -0,0 +1,10 @@
+/* { dg-additional-options "-fno-tree-ter" } */
+/* { dg-require-effective-target float16 } */
+/* { dg-add-options float16 } */
+
+int
+foo(_Float16 f, int i)
+{
+  __builtin_memcpy(&i, &f, 2);
+  return i;
+}

Reply via email to