public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [committed] Remove condition from PA indirect_jump
       [not found] ` <20150701092835.GA11111@tsaunders-iceball.corp.tor1.mozilla.com>
@ 2015-07-04 13:55   ` John David Anglin
  2015-07-05  8:09     ` Richard Sandiford
  0 siblings, 1 reply; 2+ messages in thread
From: John David Anglin @ 2015-07-04 13:55 UTC (permalink / raw)
  To: Trevor Saunders, GCC Patches; +Cc: Jeff Law, Richard Sandiford

[-- Attachment #1: Type: text/plain, Size: 317 bytes --]

The attached change removes the C condition from the PA indirect jump since it depended
on operands[] and this isn't generally allowed in named patterns.

Tested on hppa-unknown-linux-gnu and hppa2.0w-hp-hpux11.11.  Committed to trunk and active
branches.

Dave
--
John David Anglin	dave.anglin@bell.net



[-- Attachment #2: pa.md.d.txt --]
[-- Type: text/plain, Size: 694 bytes --]

2015-07-04  John David Anglin  <danglin@gcc.gnu.org>

	PR target/66114
	* config/pa/pa.md (indirect_jump): Use pmode_register_operand instead
	of register_operand.  Remove constraint.

Index: config/pa/pa.md
===================================================================
--- config/pa/pa.md	(revision 225280)
+++ config/pa/pa.md	(working copy)
@@ -6844,8 +6844,8 @@
 
 ;;; Hope this is only within a function...
 (define_insn "indirect_jump"
-  [(set (pc) (match_operand 0 "register_operand" "r"))]
-  "GET_MODE (operands[0]) == word_mode"
+  [(set (pc) (match_operand 0 "pmode_register_operand" "r"))]
+  ""
   "bv%* %%r0(%0)"
   [(set_attr "type" "branch")
    (set_attr "length" "4")])

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [committed] Remove condition from PA indirect_jump
  2015-07-04 13:55   ` [committed] Remove condition from PA indirect_jump John David Anglin
@ 2015-07-05  8:09     ` Richard Sandiford
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Sandiford @ 2015-07-05  8:09 UTC (permalink / raw)
  To: John David Anglin; +Cc: Trevor Saunders, GCC Patches, Jeff Law

John David Anglin <dave.anglin@bell.net> writes:
> The attached change removes the C condition from the PA indirect jump
> since it depended on operands[] and this isn't generally allowed in
> named patterns.
>
> Tested on hppa-unknown-linux-gnu and hppa2.0w-hp-hpux11.11.  Committed
> to trunk and active branches.

Thanks!

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-07-05  8:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <87egks2a0k.fsf@googlemail.com>
     [not found] ` <20150701092835.GA11111@tsaunders-iceball.corp.tor1.mozilla.com>
2015-07-04 13:55   ` [committed] Remove condition from PA indirect_jump John David Anglin
2015-07-05  8:09     ` Richard Sandiford

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