* [Patch, GCC]Backporting r269039 to gcc8
@ 2019-09-18 14:55 Delia Burduv
2019-10-04 15:12 ` Delia Burduv
0 siblings, 1 reply; 5+ messages in thread
From: Delia Burduv @ 2019-09-18 14:55 UTC (permalink / raw)
To: gcc-patches; +Cc: nd, law, ian, rguenther
[-- Attachment #1: Type: text/plain, Size: 994 bytes --]
Hi,
I am trying to backport r269039 to gcc8 wich solved this bug report:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86487 . I have tested it on
arm-none-linux-gnueabi,aarch64-none-linux-gnu and x86_64-pc-linux-gnu
and there was no regression. The patch applied cleanly. I don't have
commit rights, so if it is ok can someone please commit it for me?
Thanks,
Delia
gcc/ChangeLog:
2019-09-13 Delia Burduv <Delia.Burduv@arm.com>
Backport from trunk
2019-02-20 Andre Vieira <andre.simoesdiasvieira@arm.com>
PR target/86487
* lra-constraints.c(uses_hard_regs_p): Fix handling of
paradoxical SUBREGS.
gcc/testsuite/ChangeLog:
2019-09-13 Delia Burduv <Delia.Burduv@arm.com>
Backport from trunk
2019-02-20 Andre Vieira <andre.simoesdiasvieira@arm.com>
PR target/86487
* gcc.target/arm/pr86487.c: New.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: rb11823.patch --]
[-- Type: text/x-patch; name="rb11823.patch", Size: 1772 bytes --]
diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c
index 484e9fa148c32208cd3af39e3aaa944069933ac0..1dea8c959d8f0e7e2d39f0ccf1b97aa1f64b024f 100644
--- a/gcc/lra-constraints.c
+++ b/gcc/lra-constraints.c
@@ -1774,14 +1774,24 @@ uses_hard_regs_p (rtx x, HARD_REG_SET set)
return false;
code = GET_CODE (x);
mode = GET_MODE (x);
+
if (code == SUBREG)
{
+ /* For all SUBREGs we want to check whether the full multi-register
+ overlaps the set. For normal SUBREGs this means 'get_hard_regno' of
+ the inner register, for paradoxical SUBREGs this means the
+ 'get_hard_regno' of the full SUBREG and for complete SUBREGs either is
+ fine. Use the wider mode for all cases. */
+ rtx subreg = SUBREG_REG (x);
mode = wider_subreg_mode (x);
- x = SUBREG_REG (x);
- code = GET_CODE (x);
+ if (mode == GET_MODE (subreg))
+ {
+ x = subreg;
+ code = GET_CODE (x);
+ }
}
- if (REG_P (x))
+ if (REG_P (x) || SUBREG_P (x))
{
x_hard_regno = get_hard_regno (x, true);
return (x_hard_regno >= 0
diff --git a/gcc/testsuite/gcc.target/arm/pr86487.c b/gcc/testsuite/gcc.target/arm/pr86487.c
new file mode 100644
index 0000000000000000000000000000000000000000..1c1db7852d91a82a1d2b6eaa4f3d4c6dbef107f5
--- /dev/null
+++ b/gcc/testsuite/gcc.target/arm/pr86487.c
@@ -0,0 +1,10 @@
+/* { dg-skip-if "" { *-*-* } { "-march=armv[0-6]*" "-mthumb" } { "" } } */
+/* { dg-require-effective-target arm_neon_hw } */
+/* { dg-options "-O1 -mbig-endian" } */
+/* { dg-add-options arm_neon } */
+int a, b, c, d;
+long long fn1(long long p2) { return p2 == 0 ? -1 : -1 % p2; }
+void fn2(long long p1, short p2, long p3) {
+ b = fn1((d || 6) & a);
+ c = b | p3;
+}
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Patch, GCC]Backporting r269039 to gcc8
2019-09-18 14:55 [Patch, GCC]Backporting r269039 to gcc8 Delia Burduv
@ 2019-10-04 15:12 ` Delia Burduv
2019-10-04 21:27 ` Jeff Law
0 siblings, 1 reply; 5+ messages in thread
From: Delia Burduv @ 2019-10-04 15:12 UTC (permalink / raw)
To: gcc-patches; +Cc: nd, law, ian, rguenther
Ping. Has anyone had a look at the patch? Please let me know if it is fine.
Thanks,
Delia
________________________________
From: Delia Burduv <delia.burduv@arm.com>
Sent: 18 September 2019 15:54
To: gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org>
Cc: nd <nd@arm.com>; law@redhat.com <law@redhat.com>; ian@airs.com <ian@airs.com>; rguenther@suse.de <rguenther@suse.de>
Subject: [Patch, GCC]Backporting r269039 to gcc8
Hi,
I am trying to backport r269039 to gcc8 wich solved this bug report:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86487 . I have tested it on
arm-none-linux-gnueabi,aarch64-none-linux-gnu and x86_64-pc-linux-gnu
and there was no regression. The patch applied cleanly. I don't have
commit rights, so if it is ok can someone please commit it for me?
Thanks,
Delia
gcc/ChangeLog:
2019-09-13 Delia Burduv <Delia.Burduv@arm.com>
Backport from trunk
2019-02-20 Andre Vieira <andre.simoesdiasvieira@arm.com>
PR target/86487
* lra-constraints.c(uses_hard_regs_p): Fix handling of
paradoxical SUBREGS.
gcc/testsuite/ChangeLog:
2019-09-13 Delia Burduv <Delia.Burduv@arm.com>
Backport from trunk
2019-02-20 Andre Vieira <andre.simoesdiasvieira@arm.com>
PR target/86487
* gcc.target/arm/pr86487.c: New.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Patch, GCC]Backporting r269039 to gcc8
2019-10-04 15:12 ` Delia Burduv
@ 2019-10-04 21:27 ` Jeff Law
2019-10-25 17:03 ` Delia Burduv
0 siblings, 1 reply; 5+ messages in thread
From: Jeff Law @ 2019-10-04 21:27 UTC (permalink / raw)
To: Delia Burduv, gcc-patches; +Cc: nd, ian, rguenther
On 10/4/19 9:11 AM, Delia Burduv wrote:
> Ping. Has anyone had a look at the patch? Please let me know if it is fine.
I think it's fine to backport to the gcc-8 branch. Do you have commit
access?
jeff
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Patch, GCC]Backporting r269039 to gcc8
2019-10-04 21:27 ` Jeff Law
@ 2019-10-25 17:03 ` Delia Burduv
2019-11-01 22:37 ` Jeff Law
0 siblings, 1 reply; 5+ messages in thread
From: Delia Burduv @ 2019-10-25 17:03 UTC (permalink / raw)
To: Jeff Law, gcc-patches; +Cc: nd, ian, rguenther
Hello Jeff,
Yes, it is a backport to gcc-8. No, I don't have commit access. Could you please commit it for me?
Thanks,
Delia
________________________________
From: Jeff Law <law@redhat.com>
Sent: 04 October 2019 22:27
To: Delia Burduv <Delia.Burduv@arm.com>; gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org>
Cc: nd <nd@arm.com>; ian@airs.com <ian@airs.com>; rguenther@suse.de <rguenther@suse.de>
Subject: Re: [Patch, GCC]Backporting r269039 to gcc8
On 10/4/19 9:11 AM, Delia Burduv wrote:
> Ping. Has anyone had a look at the patch? Please let me know if it is fine.
I think it's fine to backport to the gcc-8 branch. Do you have commit
access?
jeff
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Patch, GCC]Backporting r269039 to gcc8
2019-10-25 17:03 ` Delia Burduv
@ 2019-11-01 22:37 ` Jeff Law
0 siblings, 0 replies; 5+ messages in thread
From: Jeff Law @ 2019-11-01 22:37 UTC (permalink / raw)
To: Delia Burduv, gcc-patches; +Cc: nd, ian, rguenther
On 10/25/19 11:02 AM, Delia Burduv wrote:
> Hello Jeff,
>
> Yes, it is a backport to gcc-8. No, I don't have commit access. Could
> you please commit it for me?
Done.
Thanks for your patience.
jeff
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-11-01 22:37 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-18 14:55 [Patch, GCC]Backporting r269039 to gcc8 Delia Burduv
2019-10-04 15:12 ` Delia Burduv
2019-10-04 21:27 ` Jeff Law
2019-10-25 17:03 ` Delia Burduv
2019-11-01 22:37 ` Jeff Law
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).