From: "Sam James" <sam@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] proj/gcc-patches:master commit in: 15.0.0/gentoo/
Date: Fri, 21 Mar 2025 06:07:31 +0000 (UTC) [thread overview]
Message-ID: <1742536989.b174b37b1211411d7b7a7d7c830da86f19d9ecb5.sam@gentoo> (raw)
commit: b174b37b1211411d7b7a7d7c830da86f19d9ecb5
Author: Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Fri Mar 21 06:03:09 2025 +0000
Commit: Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Fri Mar 21 06:03:09 2025 +0000
URL: https://gitweb.gentoo.org/proj/gcc-patches.git/commit/?id=b174b37b
15.0.0: add arm32 neon shift patch
Bug: https://gcc.gnu.org/PR117811
Signed-off-by: Sam James <sam <AT> gentoo.org>
15.0.0/gentoo/79_all_PR117811-arm-neon-shift.patch | 96 ++++++++++++++++++++++
15.0.0/gentoo/README.history | 1 +
2 files changed, 97 insertions(+)
diff --git a/15.0.0/gentoo/79_all_PR117811-arm-neon-shift.patch b/15.0.0/gentoo/79_all_PR117811-arm-neon-shift.patch
new file mode 100644
index 0000000..f2bb73a
--- /dev/null
+++ b/15.0.0/gentoo/79_all_PR117811-arm-neon-shift.patch
@@ -0,0 +1,96 @@
+https://gcc.gnu.org/bugzilla/show_bug.cgi?id=117811#c20
+
+From 5256e5419d07f7313fe661901b9a4821b3d47764 Mon Sep 17 00:00:00 2001
+From: Andrew Pinski <quic_apinski@quicinc.com>
+Date: Wed, 19 Mar 2025 16:02:12 -0700
+Subject: [PATCH] Fix expand_binop_directly in some cases [PR117811]
+
+The problem here is that when dealing with adding the equal note,
+we decide to call expand_binop directly but before we do that we
+call delete_insns_since with the last argument. The problem is that
+deletes too much and falls over. So instead we need to delete only
+up to what was done at the point of the call to expand_binop_directly
+
+Bootstrapped and tested on x86_64-linux-gnu. Also Christophe Lyon tested
+it on arm-linux-gnueabihf.
+
+ PR middle-end/117811
+
+gcc/ChangeLog:
+
+ * optabs.cc (expand_binop_directly): Delete only what was done
+ since the entry before calling expand_binop.
+
+gcc/testsuite/ChangeLog:
+
+ * c-c++-common/torture/pr117811-1.c: New test.
+
+Signed-off-by: Andrew Pinski <quic_apinski@quicinc.com>
+---
+ gcc/optabs.cc | 3 +-
+ .../c-c++-common/torture/pr117811-1.c | 32 +++++++++++++++++++
+ 2 files changed, 34 insertions(+), 1 deletion(-)
+ create mode 100644 gcc/testsuite/c-c++-common/torture/pr117811-1.c
+
+diff --git a/gcc/optabs.cc b/gcc/optabs.cc
+index 36f2e6af8b5..366df985147 100644
+--- a/gcc/optabs.cc
++++ b/gcc/optabs.cc
+@@ -1380,6 +1380,7 @@ expand_binop_directly (enum insn_code icode, machine_mode mode, optab binoptab,
+ rtx_insn *pat;
+ rtx xop0 = op0, xop1 = op1;
+ bool canonicalize_op1 = false;
++ rtx_insn *entry_last = get_last_insn ();
+
+ /* If it is a commutative operator and the modes would match
+ if we would swap the operands, we can save the conversions. */
+@@ -1466,7 +1467,7 @@ expand_binop_directly (enum insn_code icode, machine_mode mode, optab binoptab,
+ optab_to_code (binoptab),
+ ops[1].value, ops[2].value, mode0))
+ {
+- delete_insns_since (last);
++ delete_insns_since (entry_last);
+ return expand_binop (mode, binoptab, op0, op1, NULL_RTX,
+ unsignedp, methods);
+ }
+diff --git a/gcc/testsuite/c-c++-common/torture/pr117811-1.c b/gcc/testsuite/c-c++-common/torture/pr117811-1.c
+new file mode 100644
+index 00000000000..a0558cbd0ae
+--- /dev/null
++++ b/gcc/testsuite/c-c++-common/torture/pr117811-1.c
+@@ -0,0 +1,32 @@
++/* { dg-do run } */
++
++/* PR middle-end/117811 */
++/* Tests that right shift on vectors work. */
++
++typedef int v4 __attribute__((vector_size(4*sizeof(int))));
++
++void buggy_shift( v4 *vec, int shift ) __attribute__((noipa));
++void buggy_shift( v4 *vec, int shift )
++{
++ v4 t = *vec;
++
++ if ( shift > 0 )
++ {
++ t = t >> shift;
++ }
++
++ *vec = t;
++}
++
++int main()
++{
++ v4 v = {0x0<<4, 0x1<<4, 2<<4, 3<<4};
++ v4 r = {0x0, 0x1, 2, 3};
++ int shift = 4;
++ buggy_shift(&v, shift);
++ for(int i = 0; i < 4; i++)
++ if (v[i] != r[i])
++ __builtin_abort();
++ return 0;
++}
++
+--
+2.43.0
+
diff --git a/15.0.0/gentoo/README.history b/15.0.0/gentoo/README.history
index f55d569..c3b9934 100644
--- a/15.0.0/gentoo/README.history
+++ b/15.0.0/gentoo/README.history
@@ -2,6 +2,7 @@
+ 77_all_PR119376-Disable-clang-musttail.patch
+ 78_all_PR118615.patch
+ + 79_all_PR117811-arm-neon-shift.patch
47 16 March 2025
next reply other threads:[~2025-03-21 6:07 UTC|newest]
Thread overview: 183+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-21 6:07 Sam James [this message]
-- strict thread matches above, loose matches on Subject: below --
2025-03-25 10:27 [gentoo-commits] proj/gcc-patches:master commit in: 15.0.0/gentoo/ Sam James
2025-03-25 8:38 Sam James
2025-03-25 2:32 Sam James
2025-03-25 1:27 Sam James
2025-03-24 0:35 Sam James
2025-03-21 19:31 Sam James
2025-03-21 17:21 Sam James
2025-03-21 16:23 Sam James
2025-03-21 11:20 Sam James
2025-03-21 8:51 Sam James
2025-03-20 22:08 Sam James
2025-03-20 1:59 Sam James
2025-03-20 1:59 Sam James
2025-03-16 22:37 Sam James
2025-03-14 14:46 Sam James
2025-03-14 13:37 Sam James
2025-03-13 16:48 Sam James
2025-03-13 10:08 Sam James
2025-03-11 10:32 Sam James
2025-03-07 16:54 Sam James
2025-03-03 16:38 Sam James
2025-03-01 10:33 Sam James
2025-03-01 6:50 Sam James
2025-02-17 1:30 Sam James
2025-02-13 9:23 Sam James
2025-02-12 15:12 Sam James
2025-02-10 21:22 Sam James
2025-02-09 23:58 Sam James
2025-02-07 23:37 Sam James
2025-02-07 21:19 Sam James
2025-02-03 22:04 Sam James
2025-02-02 22:41 Sam James
2025-01-29 20:21 Sam James
2025-01-26 22:52 Sam James
2025-01-22 16:27 Sam James
2025-01-19 22:43 Sam James
2025-01-16 23:11 Sam James
2025-01-16 23:11 Sam James
2025-01-15 11:41 Sam James
2025-01-14 16:22 Sam James
2025-01-14 15:06 Sam James
2025-01-14 15:06 Sam James
2025-01-14 12:29 Sam James
2025-01-14 8:43 Sam James
2025-01-14 8:40 Sam James
2025-01-13 13:58 Sam James
2025-01-13 6:00 Sam James
2025-01-13 3:40 Sam James
2025-01-13 3:23 Sam James
2025-01-13 3:20 Sam James
2025-01-13 0:20 Sam James
2025-01-12 18:53 Sam James
2025-01-11 12:53 Sam James
2025-01-08 21:51 Sam James
2025-01-06 10:50 Sam James
2025-01-06 10:03 Sam James
2025-01-06 4:49 Sam James
2025-01-06 4:44 Sam James
2025-01-06 4:13 Sam James
2025-01-06 4:13 Sam James
2025-01-06 4:13 Sam James
2025-01-06 4:03 Sam James
2025-01-05 23:19 Sam James
2025-01-03 3:07 Sam James
2024-12-30 1:05 Sam James
2024-12-29 10:00 Sam James
2024-12-27 15:14 Sam James
2024-12-24 20:48 Sam James
2024-12-22 22:46 Sam James
2024-12-20 11:25 Sam James
2024-12-20 5:57 Sam James
2024-12-20 1:55 Sam James
2024-12-19 18:34 Sam James
2024-12-13 13:23 Sam James
2024-12-13 11:52 Sam James
2024-12-13 5:08 Sam James
2024-12-12 12:28 Sam James
2024-12-11 4:41 Sam James
2024-12-11 0:58 Sam James
2024-12-10 19:19 Sam James
2024-12-10 14:55 Sam James
2024-12-10 5:19 Sam James
2024-12-10 5:13 Sam James
2024-12-10 5:11 Sam James
2024-12-10 5:07 Sam James
2024-12-09 3:05 Sam James
2024-12-08 22:41 Sam James
2024-12-06 17:33 Sam James
2024-12-04 20:40 Sam James
2024-12-01 22:51 Sam James
2024-12-01 22:51 Sam James
2024-11-30 11:30 Sam James
2024-11-27 17:42 Sam James
2024-11-25 15:10 Sam James
2024-11-25 3:01 Sam James
2024-11-25 3:00 Sam James
2024-11-25 3:00 Sam James
2024-11-24 22:42 Sam James
2024-11-18 17:25 Sam James
2024-11-18 10:42 Sam James
2024-11-18 10:42 Sam James
2024-11-18 9:25 Sam James
2024-11-18 9:25 Sam James
2024-11-14 18:38 Sam James
2024-11-13 4:26 Sam James
2024-11-13 0:16 Sam James
2024-11-12 2:33 Sam James
2024-11-11 19:46 Sam James
2024-11-11 19:46 Sam James
2024-11-10 22:41 Sam James
2024-11-09 16:24 Sam James
2024-11-09 7:55 Sam James
2024-11-08 8:22 Sam James
2024-11-07 16:13 Sam James
2024-11-03 23:16 Sam James
2024-11-01 8:24 Sam James
2024-11-01 8:24 Sam James
2024-11-01 8:18 Sam James
2024-11-01 8:17 Sam James
2024-10-30 16:03 Sam James
2024-10-29 19:17 Sam James
2024-10-28 21:32 Sam James
2024-10-28 8:09 Sam James
2024-10-23 15:40 Sam James
2024-10-22 19:09 Sam James
2024-10-22 18:34 Sam James
2024-10-21 12:33 Sam James
2024-10-21 12:27 Sam James
2024-10-21 12:26 Sam James
2024-10-21 11:45 Sam James
2024-10-20 22:42 Sam James
2024-10-18 14:05 Sam James
2024-10-18 10:35 Sam James
2024-10-17 23:33 Sam James
2024-10-17 23:03 Sam James
2024-10-17 5:01 Sam James
2024-10-17 4:15 Sam James
2024-10-13 22:48 Sam James
2024-10-07 2:45 Sam James
2024-10-04 10:37 Sam James
2024-10-04 9:28 Sam James
2024-10-02 19:45 Sam James
2024-09-30 14:05 Sam James
2024-09-29 22:56 Sam James
2024-09-24 1:41 Sam James
2024-09-23 15:23 Sam James
2024-09-02 2:28 Sam James
2024-08-26 13:44 Sam James
2024-08-26 6:24 Sam James
2024-08-23 13:51 Sam James
2024-08-20 20:31 Sam James
2024-08-19 18:43 Sam James
2024-08-14 9:48 Sam James
2024-08-14 2:57 Sam James
2024-08-11 22:40 Sam James
2024-08-09 19:54 Sam James
2024-08-09 19:54 Sam James
2024-08-09 19:47 Sam James
2024-08-09 19:25 Sam James
2024-08-08 11:10 Sam James
2024-08-08 11:06 Sam James
2024-08-08 11:03 Sam James
2024-08-05 9:09 Sam James
2024-08-05 1:54 Sam James
2024-08-05 1:51 Sam James
2024-08-02 20:39 Sam James
2024-08-01 14:40 Sam James
2024-07-28 23:34 Sam James
2024-07-22 1:11 Sam James
2024-07-19 11:14 Sam James
2024-07-18 0:45 Sam James
2024-07-14 23:36 Sam James
2024-06-28 12:49 Sam James
2024-06-27 0:02 Sam James
2024-06-26 23:57 Sam James
2024-06-16 22:45 Sam James
2024-06-10 20:18 Sam James
2024-06-10 17:28 Sam James
2024-06-10 17:28 Sam James
2024-06-10 2:08 Sam James
2024-06-08 17:03 Sam James
2024-06-08 17:03 Sam James
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1742536989.b174b37b1211411d7b7a7d7c830da86f19d9ecb5.sam@gentoo \
--to=sam@gentoo.org \
--cc=gentoo-commits@lists.gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox