Commit: RX: Fix simple_return pattern

View: New views
3 Messages — Rating Filter:   Alert me  

Commit: RX: Fix simple_return pattern

by Nick Clifton :: Rate this Message:

| View Threaded | Show Only this Message

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@...>

        * config/rx/rx.md (simple_return): Use the simple_return rtl.

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")

Re: Commit: RX: Fix simple_return pattern

by Mike Stump-3 :: Rate this Message:

| View Threaded | Show Only this Message

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?

Re: Commit: RX: Fix simple_return pattern

by Nick Clifton :: Rate this Message:

| View Threaded | Show Only this Message

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