Patchwork Commit: RX: Fix simple_return pattern

login
register
mail settings
Submitter Nick Clifton
Date June 26, 2012, 8:59 a.m.
Message ID <87ipeeih3s.fsf@redhat.com>
Download mbox | patch
Permalink /patch/167347/
State New
Headers show

Comments

Nick Clifton - June 26, 2012, 8:59 a.m.
Hi Guys,

  I am applying the patch below to the 4.7 branch.  It fixes the
  simple_return pattern in the RX backend so that it uses the
  (simple_return) rtl.  This fixes 59 gcc testsuite failures and
  introduces no regressions.

  I plan on applying a similar patch to the mainline sources once I have
  finished regression testing them.

Cheers
  Nick
  
gcc/ChangeLog
2012-06-26  Nick Clifton  <nickc@redhat.com>

	* config/rx/rx.md (simple_return): Use the simple_return rtl.
Mike Stump - June 26, 2012, 7:52 p.m.
On Jun 26, 2012, at 1:59 AM, Nick Clifton wrote:
>  I am applying the patch below to the 4.7 branch.  It fixes the
>  simple_return pattern in the RX backend so that it uses the
>  (simple_return) rtl.  This fixes 59 gcc testsuite failures and
>  introduces no regressions.
> 
>  I plan on applying a similar patch to the mainline sources once I have
>  finished regression testing them.

Really, trunk should always go in first...  Could you hold 4.7 until trunk goes in?
Nick Clifton - June 27, 2012, 7:24 a.m.
Hi Mike,
>>   I plan on applying a similar patch to the mainline sources once I have
>>   finished regression testing them.
>
> Really, trunk should always go in first...  Could you hold 4.7 until trunk goes in?

Sorry, I had already checked the patch in.  I have now checked in the 
trunk patch, with only 1 day between the two patches going in.  In the 
future I'll be sure to patch the trunk first.

Cheers
   Nick

Patch

Index: gcc/config/rx/rx.md
===================================================================
--- gcc/config/rx/rx.md	(revision 188932)
+++ gcc/config/rx/rx.md	(working copy)
@@ -348,7 +348,7 @@ 
 )
 
 (define_insn "simple_return"
-  [(return)]
+  [(simple_return)]
   ""
   "rts"
   [(set_attr "length" "1")