Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package gcc13 for openSUSE:Factory checked
in at 2023-09-28 00:24:24
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/gcc13 (Old)
and /work/SRC/openSUSE:Factory/.gcc13.new.23327 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "gcc13"
Thu Sep 28 00:24:24 2023 rev:16 rq:1113437 version:13.2.1+git7813
Changes:
--------
--- /work/SRC/openSUSE:Factory/gcc13/gcc13.changes 2023-09-14
16:24:20.623748692 +0200
+++ /work/SRC/openSUSE:Factory/.gcc13.new.23327/gcc13.changes 2023-09-28
00:32:59.504908051 +0200
@@ -1,0 +2,5 @@
+Fri Sep 22 08:07:30 UTC 2023 - Dirk Müller <dmuel...@suse.com>
+
+- add pr111411.patch (bsc#1215427)
+
+-------------------------------------------------------------------
New:
----
pr111411.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ cross-aarch64-gcc13-bootstrap.spec ++++++
--- /var/tmp/diff_new_pack.25zRUh/_old 2023-09-28 00:33:04.685095033 +0200
+++ /var/tmp/diff_new_pack.25zRUh/_new 2023-09-28 00:33:04.689095177 +0200
@@ -134,6 +134,7 @@
Patch19: gcc11-gdwarf-4-default.patch
Patch20: gcc11-amdgcn-disable-hot-cold-partitioning.patch
Patch21: gdcflags.patch
+Patch22: pr111411.patch
# A set of patches from the RH srpm
Patch51: gcc41-ppc32-retaddr.patch
# Some patches taken from Debian
@@ -353,6 +354,7 @@
%patch19 -p1
%endif
%patch21 -p1
+%patch22 -p1
%patch51
%patch60 -p1
%patch61 -p1
cross-aarch64-gcc13.spec: same change
cross-amdgcn-gcc13.spec: same change
cross-arm-gcc13.spec: same change
cross-arm-none-gcc13-bootstrap.spec: same change
cross-arm-none-gcc13.spec: same change
cross-avr-gcc13-bootstrap.spec: same change
cross-avr-gcc13.spec: same change
cross-bpf-gcc13.spec: same change
cross-epiphany-gcc13-bootstrap.spec: same change
cross-epiphany-gcc13.spec: same change
cross-hppa-gcc13-bootstrap.spec: same change
cross-hppa-gcc13.spec: same change
cross-m68k-gcc13.spec: same change
cross-mips-gcc13.spec: same change
cross-nvptx-gcc13.spec: same change
cross-ppc64-gcc13.spec: same change
cross-ppc64le-gcc13-bootstrap.spec: same change
cross-ppc64le-gcc13.spec: same change
cross-pru-gcc13-bootstrap.spec: same change
cross-pru-gcc13.spec: same change
cross-riscv64-elf-gcc13-bootstrap.spec: same change
cross-riscv64-elf-gcc13.spec: same change
cross-riscv64-gcc13-bootstrap.spec: same change
cross-riscv64-gcc13.spec: same change
cross-rx-gcc13-bootstrap.spec: same change
cross-rx-gcc13.spec: same change
cross-s390x-gcc13-bootstrap.spec: same change
cross-s390x-gcc13.spec: same change
cross-sparc-gcc13.spec: same change
cross-sparc64-gcc13.spec: same change
cross-x86_64-gcc13.spec: same change
gcc13-testresults.spec: same change
gcc13.spec: same change
++++++ gcc.spec.in ++++++
--- /var/tmp/diff_new_pack.25zRUh/_old 2023-09-28 00:33:05.709131996 +0200
+++ /var/tmp/diff_new_pack.25zRUh/_new 2023-09-28 00:33:05.713132140 +0200
@@ -386,6 +386,7 @@
Patch19: gcc11-gdwarf-4-default.patch
Patch20: gcc11-amdgcn-disable-hot-cold-partitioning.patch
Patch21: gdcflags.patch
+Patch22: pr111411.patch
# A set of patches from the RH srpm
Patch51: gcc41-ppc32-retaddr.patch
# Some patches taken from Debian
@@ -1350,6 +1351,7 @@
%patch19 -p1
%endif
%patch21 -p1
+%patch22 -p1
%patch51
%patch60 -p1
%patch61 -p1
++++++ pr111411.patch ++++++
>From 74f99f1adc696f446115f36974a3f94f66294a53 Mon Sep 17 00:00:00 2001
From: Richard Sandiford <richard.sandif...@arm.com>
Date: Wed, 20 Sep 2023 11:13:20 +0100
Subject: [PATCH] aarch64: Fix loose ldpstp check [PR111411]
aarch64_operands_ok_for_ldpstp contained the code:
/* One of the memory accesses must be a mempair operand.
If it is not the first one, they need to be swapped by the
peephole. */
if (!aarch64_mem_pair_operand (mem_1, GET_MODE (mem_1))
&& !aarch64_mem_pair_operand (mem_2, GET_MODE (mem_2)))
return false;
But the requirement isn't just that one of the accesses must be a
valid mempair operand. It's that the lower access must be, since
that's the access that will be used for the instruction operand.
gcc/
PR target/111411
* config/aarch64/aarch64.cc (aarch64_operands_ok_for_ldpstp): Require
the lower memory access to a mem-pair operand.
gcc/testsuite/
PR target/111411
* gcc.dg/rtl/aarch64/pr111411.c: New test.
(cherry picked from commit 2d38f45bcca62ca0c7afef4b579f82c5c2a01610)
---
gcc/config/aarch64/aarch64.cc | 8 ++-
gcc/testsuite/gcc.dg/rtl/aarch64/pr111411.c | 57 +++++++++++++++++++++
2 files changed, 60 insertions(+), 5 deletions(-)
create mode 100644 gcc/testsuite/gcc.dg/rtl/aarch64/pr111411.c
diff --git a/gcc/config/aarch64/aarch64.cc b/gcc/config/aarch64/aarch64.cc
index 96c3f48fdc4..a979accd90a 100644
--- a/gcc/config/aarch64/aarch64.cc
+++ b/gcc/config/aarch64/aarch64.cc
@@ -26031,11 +26031,9 @@ aarch64_operands_ok_for_ldpstp (rtx *operands, bool
load,
gcc_assert (known_eq (GET_MODE_SIZE (GET_MODE (mem_1)),
GET_MODE_SIZE (GET_MODE (mem_2))));
- /* One of the memory accesses must be a mempair operand.
- If it is not the first one, they need to be swapped by the
- peephole. */
- if (!aarch64_mem_pair_operand (mem_1, GET_MODE (mem_1))
- && !aarch64_mem_pair_operand (mem_2, GET_MODE (mem_2)))
+ /* The lower memory access must be a mem-pair operand. */
+ rtx lower_mem = reversed ? mem_2 : mem_1;
+ if (!aarch64_mem_pair_operand (lower_mem, GET_MODE (lower_mem)))
return false;
if (REG_P (reg_1) && FP_REGNUM_P (REGNO (reg_1)))
diff --git a/gcc/testsuite/gcc.dg/rtl/aarch64/pr111411.c
b/gcc/testsuite/gcc.dg/rtl/aarch64/pr111411.c
new file mode 100644
index 00000000000..ad07e9c6c89
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/rtl/aarch64/pr111411.c
@@ -0,0 +1,57 @@
+/* { dg-do compile { target aarch64*-*-* } } */
+/* { dg-require-effective-target lp64 } */
+/* { dg-options "-O -fdisable-rtl-postreload -fpeephole2 -fno-schedule-fusion"
} */
+
+extern int data[];
+
+void __RTL (startwith ("ira")) foo (void *ptr)
+{
+ (function "foo"
+ (param "ptr"
+ (DECL_RTL (reg/v:DI <0> [ ptr ]))
+ (DECL_RTL_INCOMING (reg/v:DI x0 [ ptr ]))
+ ) ;; param "ptr"
+ (insn-chain
+ (block 2
+ (edge-from entry (flags "FALLTHRU"))
+ (cnote 3 [bb 2] NOTE_INSN_BASIC_BLOCK)
+ (insn 4 (set (reg:DI <0>) (reg:DI x0)))
+ (insn 5 (set (reg:DI <1>)
+ (plus:DI (reg:DI <0>) (const_int 768))))
+ (insn 6 (set (mem:SI (plus:DI (reg:DI <0>)
+ (const_int 508)) [1 &data+508 S4 A4])
+ (const_int 0)))
+ (insn 7 (set (mem:SI (plus:DI (reg:DI <1>)
+ (const_int -256)) [1 &data+512 S4 A4])
+ (const_int 0)))
+ (edge-to exit (flags "FALLTHRU"))
+ ) ;; block 2
+ ) ;; insn-chain
+ ) ;; function
+}
+
+void __RTL (startwith ("ira")) bar (void *ptr)
+{
+ (function "bar"
+ (param "ptr"
+ (DECL_RTL (reg/v:DI <0> [ ptr ]))
+ (DECL_RTL_INCOMING (reg/v:DI x0 [ ptr ]))
+ ) ;; param "ptr"
+ (insn-chain
+ (block 2
+ (edge-from entry (flags "FALLTHRU"))
+ (cnote 3 [bb 2] NOTE_INSN_BASIC_BLOCK)
+ (insn 4 (set (reg:DI <0>) (reg:DI x0)))
+ (insn 5 (set (reg:DI <1>)
+ (plus:DI (reg:DI <0>) (const_int 768))))
+ (insn 6 (set (mem:SI (plus:DI (reg:DI <1>)
+ (const_int -256)) [1 &data+512 S4 A4])
+ (const_int 0)))
+ (insn 7 (set (mem:SI (plus:DI (reg:DI <0>)
+ (const_int 508)) [1 &data+508 S4 A4])
+ (const_int 0)))
+ (edge-to exit (flags "FALLTHRU"))
+ ) ;; block 2
+ ) ;; insn-chain
+ ) ;; function
+}
--
2.39.3
