[RX] Fix PR 81819

Message ID 1515684556.12491.20.camel@t-online.de
State New
Headers show
Series
  • [RX] Fix PR 81819
Related show

Commit Message

Oleg Endo Jan. 11, 2018, 3:29 p.m.
Hi,

The attached patch fixes PR 81819, which popped up on GCC 7.  I assume
it's also there on trunk, but can't build my app with trunk compiler
because it's got other issues.

Unfortunately I can't add the reproducer test case since it happens
only when building a bigger app with LTO.  But I have confirmed that
with this fix the app builds and runs.

OK for trunk and GCC 7?

Cheers,
Oleg

gcc/ChangeLog:
	PR target/81819
	* config/rx/rx.c (rx_is_restricted_memory_address):
	Handle SUBREG case.

Comments

Nick Clifton Jan. 11, 2018, 5:32 p.m. | #1
Hi Oleg,

> gcc/ChangeLog:
> 	PR target/81819
> 	* config/rx/rx.c (rx_is_restricted_memory_address):
> 	Handle SUBREG case.

Go ahead make my day^H^H^H^H^H^H

Approved - please apply.

Cheers
  Nick
Oleg Endo Jan. 13, 2018, 8:14 a.m. | #2
On Thu, 2018-01-11 at 17:32 +0000, Nick Clifton wrote:

> > gcc/ChangeLog:
> > 	PR target/81819
> > 	* config/rx/rx.c (rx_is_restricted_memory_address):
> > 	Handle SUBREG case.
> Go ahead make my day^H^H^H^H^H^H
> 
> Approved - please apply.

Committed as r256578 (trunk) and r256579 (GCC 7 branch).

Cheers,
Oleg

Patch

Index: gcc/config/rx/rx.c
===================================================================
--- gcc/config/rx/rx.c	(revision 256385)
+++ gcc/config/rx/rx.c	(working copy)
@@ -284,6 +284,9 @@ 
       /* Simple memory addresses are OK.  */
       return true;
 
+    case SUBREG:
+      return RX_REG_P (SUBREG_REG (mem));
+
     case PRE_DEC:
     case POST_INC:
       return false;