public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH,ARM] Set attribute predicable
@ 2013-02-14 13:54 Greta Yorsh
  2013-02-14 15:57 ` Richard Earnshaw
  0 siblings, 1 reply; 2+ messages in thread
From: Greta Yorsh @ 2013-02-14 13:54 UTC (permalink / raw)
  To: GCC Patches; +Cc: Ramana Radhakrishnan, Richard Earnshaw

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

This patch sets attribute "predicable" to "yes" for patterns that handle add
with carry and already use %? in their output statements.

Ok for trunk?
Thanks,
Greta

gcc/

2013-02-14  Greta Yorsh  <Greta.Yorsh@arm.com>

	* config/arm/arm.md (addsi3_carryin_<optab>): Set attribute
	"predicable" to yes.
	(addsi3_carryin_alt2_<optab>,addsi3_carryin_shift_<optab>):
Likewise.

[-- Attachment #2: predicable-attr.patch.txt --]
[-- Type: text/plain, Size: 1090 bytes --]

diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md
index 1cb1515b1fa57c6052b68eb8701616c1b80e7416..35294dd6560ac63279d95eca6cf774257e06bd93 100644
--- a/gcc/config/arm/arm.md
+++ b/gcc/config/arm/arm.md
@@ -974,7 +974,8 @@ (define_insn "*addsi3_carryin_<optab>"
   "@
    adc%?\\t%0, %1, %2
    sbc%?\\t%0, %1, #%B2"
-  [(set_attr "conds" "use")]
+  [(set_attr "conds" "use")
+   (set_attr "predicable" "yes")]
 )
 
 (define_insn "*addsi3_carryin_alt2_<optab>"
@@ -986,7 +987,8 @@ (define_insn "*addsi3_carryin_alt2_<opta
   "@
    adc%?\\t%0, %1, %2
    sbc%?\\t%0, %1, #%B2"
-  [(set_attr "conds" "use")]
+  [(set_attr "conds" "use")
+   (set_attr "predicable" "yes")]
 )
 
 (define_insn "*addsi3_carryin_shift_<optab>"
@@ -1000,6 +1002,7 @@ (define_insn "*addsi3_carryin_shift_<opt
   "TARGET_32BIT"
   "adc%?\\t%0, %1, %3%S2"
   [(set_attr "conds" "use")
+   (set_attr "predicable" "yes")
    (set (attr "type") (if_then_else (match_operand 4 "const_int_operand" "")
 		      (const_string "alu_shift")
 		      (const_string "alu_shift_reg")))]

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

* Re: [PATCH,ARM] Set attribute predicable
  2013-02-14 13:54 [PATCH,ARM] Set attribute predicable Greta Yorsh
@ 2013-02-14 15:57 ` Richard Earnshaw
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Earnshaw @ 2013-02-14 15:57 UTC (permalink / raw)
  To: Greta Yorsh; +Cc: GCC Patches, Ramana Radhakrishnan

On 14/02/13 13:53, Greta Yorsh wrote:
> This patch sets attribute "predicable" to "yes" for patterns that handle add
> with carry and already use %? in their output statements.
>
> Ok for trunk?
> Thanks,
> Greta
>
> gcc/
>
> 2013-02-14  Greta Yorsh  <Greta.Yorsh@arm.com>
>
> 	* config/arm/arm.md (addsi3_carryin_<optab>): Set attribute
> 	"predicable" to yes.
> 	(addsi3_carryin_alt2_<optab>,addsi3_carryin_shift_<optab>):
> Likewise.
>
>

OK when stage1 opens.

R.


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

end of thread, other threads:[~2013-02-14 15:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-14 13:54 [PATCH,ARM] Set attribute predicable Greta Yorsh
2013-02-14 15:57 ` Richard Earnshaw

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