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.

diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
index b10059e4f5808b42e97dd6c4d5dd32c913fcf242..03188a64ab17396f26e752b08b75a9f6dd21722d 100644 (file)
--- a/gcc/config/aarch64/aarch64.md
+++ b/gcc/config/aarch64/aarch64.md
@@ -6435,8 +6435,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 (file)
index 0000000..78cdda6
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/torture/pr119133.c
@@ -0,0 +1,8 @@
+/* { dg-additional-options "-fno-tree-ter" } */
+
+int
+foo(_Float16 f, int i)
+{
+  __builtin_memcpy(&i, &f, 2);
+  return i;
+}