]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Fix ICE in rtl check due to CONST_WIDE_INT in CONST_VECTOR_DUPLICATE_P
authorliuhongt <hongtao.liu@intel.com>
Tue, 11 Jun 2024 02:23:27 +0000 (10:23 +0800)
committerliuhongt <hongtao.liu@intel.com>
Wed, 12 Jun 2024 00:16:39 +0000 (08:16 +0800)
The patch add extra check to make sure the component of CONST_VECTOR
is CONST_INT_P.

gcc/ChangeLog:

PR target/115384
* simplify-rtx.cc (simplify_context::simplify_binary_operation_1):
Only do the simplification of (AND (ASHIFTRT A imm) mask)
to (LSHIFTRT A imm) when the component of const_vector is
CONST_INT_P.

gcc/testsuite/ChangeLog:

* gcc.target/i386/pr115384.c: New test.

gcc/simplify-rtx.cc
gcc/testsuite/gcc.target/i386/pr115384.c [new file with mode: 0644]

index 9bc3ef9ad9fda22ec376822a1cce5ffa1a7e6ddd..3ee95f74d3dbb567e23a7cc3806cf4f370ee33a4 100644 (file)
@@ -4072,9 +4072,11 @@ simplify_context::simplify_binary_operation_1 (rtx_code code,
       if (VECTOR_MODE_P (mode) && GET_CODE (op0) == ASHIFTRT
          && (CONST_INT_P (XEXP (op0, 1))
              || (GET_CODE (XEXP (op0, 1)) == CONST_VECTOR
-                 && CONST_VECTOR_DUPLICATE_P (XEXP (op0, 1))))
+                 && CONST_VECTOR_DUPLICATE_P (XEXP (op0, 1))
+                 && CONST_INT_P (XVECEXP (XEXP (op0, 1), 0, 0))))
          && GET_CODE (op1) == CONST_VECTOR
-         && CONST_VECTOR_DUPLICATE_P (op1))
+         && CONST_VECTOR_DUPLICATE_P (op1)
+         && CONST_INT_P (XVECEXP (op1, 0, 0)))
        {
          unsigned HOST_WIDE_INT shift_count
            = (CONST_INT_P (XEXP (op0, 1))
diff --git a/gcc/testsuite/gcc.target/i386/pr115384.c b/gcc/testsuite/gcc.target/i386/pr115384.c
new file mode 100644 (file)
index 0000000..31dd6f4
--- /dev/null
@@ -0,0 +1,12 @@
+/* { dg-do compile { target int128 } } */
+/* { dg-options "-O" } */
+
+typedef __attribute__((__vector_size__(sizeof(__int128)))) __int128 W;
+
+W w;
+
+void
+foo()
+{
+  w = w >> 4 & 18446744073709551600llu;
+}