* [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 @ 2012-11-19 17:51 Kyrylo Tkachov 2012-11-19 18:05 ` Ramana Radhakrishnan 2012-11-20 9:17 ` Richard Earnshaw 0 siblings, 2 replies; 6+ messages in thread From: Kyrylo Tkachov @ 2012-11-19 17:51 UTC (permalink / raw) To: gcc-patches; +Cc: Richard Earnshaw, Ramana Radhakrishnan [-- Attachment #1: Type: text/plain, Size: 612 bytes --] Hi all, This patch updates the arm_abssi2 and arm_neg_abssi2 patterns in the ARM machine description. We define the predicable attribute based on the alternative. When the patterns were introduced it was not possible to do that. Now the second alternative in each of the patterns that supports predication is defined as predicable. No regressions on arm-none-eabi with qemu. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-11-19 Kyrylo Tkachov <kyrylo.tkachov@arm.com> * config/arm/arm.md (*arm_abssi2): Define predicable attribute. (*arm_neg_abssi2): Define predicable attribute. [-- Attachment #2: abssi_predicable.txt --] [-- Type: text/plain, Size: 698 bytes --] --- a/gcc/config/arm/arm.md +++ b/gcc/config/arm/arm.md @@ -4263,7 +4263,7 @@ (define_insn "*arm_abssi2" eor%?\\t%0, %1, %1, asr #31\;sub%?\\t%0, %0, %1, asr #31" [(set_attr "conds" "clob,*") (set_attr "shift" "1") - ;; predicable can't be set based on the variant, so left as no + (set_attr "predicable" "no, yes") (set_attr "length" "8")] ) @@ -4291,7 +4291,7 @@ (define_insn "*arm_neg_abssi2" eor%?\\t%0, %1, %1, asr #31\;rsb%?\\t%0, %0, %1, asr #31" [(set_attr "conds" "clob,*") (set_attr "shift" "1") - ;; predicable can't be set based on the variant, so left as no + (set_attr "predicable" "no, yes") (set_attr "length" "8")] ) ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 2012-11-19 17:51 [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 Kyrylo Tkachov @ 2012-11-19 18:05 ` Ramana Radhakrishnan 2012-11-20 10:25 ` Kyrylo Tkachov 2012-11-20 9:17 ` Richard Earnshaw 1 sibling, 1 reply; 6+ messages in thread From: Ramana Radhakrishnan @ 2012-11-19 18:05 UTC (permalink / raw) To: Kyrylo Tkachov; +Cc: gcc-patches, Richard Earnshaw On 11/19/12 17:51, Kyrylo Tkachov wrote: > Hi all, > This patch updates the arm_abssi2 and arm_neg_abssi2 patterns in the ARM > machine description. > We define the predicable attribute based on the alternative. When the > patterns were introduced it was not possible to do that. > Now the second alternative in each of the patterns that supports predication > is defined as predicable. > > No regressions on arm-none-eabi with qemu. > > Ok for trunk? Ok, please apply. Ramana ^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 2012-11-19 18:05 ` Ramana Radhakrishnan @ 2012-11-20 10:25 ` Kyrylo Tkachov 2012-11-20 10:32 ` Richard Earnshaw 0 siblings, 1 reply; 6+ messages in thread From: Kyrylo Tkachov @ 2012-11-20 10:25 UTC (permalink / raw) To: gcc-patches; +Cc: Ramana Radhakrishnan, Richard Earnshaw [-- Attachment #1: Type: text/plain, Size: 1442 bytes --] Hi all, This is the updated version of the patch. It defines the predicable attribute for the equivalent abssi2 and neg_abssi2 patterns in thumb2.md. The previous version (with the changes only to arm.md) has been okayed but not applied and this patch supersedes it. The previous version is: http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01578.html No regressions for arm-none-eabi for qemu. Ok for trunk? Thanks, Kyrill gcc/ChangeLog 2012-11-20 Kyrylo Tkachov <kyrylo.tkachov@arm.com> * config/arm/arm.md (*arm_abssi2): Define predicable attribute. (*arm_neg_abssi2): Likewise. * config/arm/thumb2.md (*thumb2_abssi2): Likewise. (*thumb2_neg_abssi2): Likewise. -----Original Message----- From: Ramana Radhakrishnan Sent: 19 November 2012 18:05 To: Kyrylo Tkachov Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw Subject: Re: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 On 11/19/12 17:51, Kyrylo Tkachov wrote: > Hi all, > This patch updates the arm_abssi2 and arm_neg_abssi2 patterns in the ARM > machine description. > We define the predicable attribute based on the alternative. When the > patterns were introduced it was not possible to do that. > Now the second alternative in each of the patterns that supports predication > is defined as predicable. > > No regressions on arm-none-eabi with qemu. > > Ok for trunk? Ok, please apply. Ramana [-- Attachment #2: abssi_predicable.txt --] [-- Type: text/plain, Size: 1573 bytes --] --- a/gcc/config/arm/arm.md +++ b/gcc/config/arm/arm.md @@ -4263,7 +4263,7 @@ (define_insn "*arm_abssi2" eor%?\\t%0, %1, %1, asr #31\;sub%?\\t%0, %0, %1, asr #31" [(set_attr "conds" "clob,*") (set_attr "shift" "1") - ;; predicable can't be set based on the variant, so left as no + (set_attr "predicable" "no, yes") (set_attr "length" "8")] ) @@ -4291,7 +4291,7 @@ (define_insn "*arm_neg_abssi2" eor%?\\t%0, %1, %1, asr #31\;rsb%?\\t%0, %0, %1, asr #31" [(set_attr "conds" "clob,*") (set_attr "shift" "1") - ;; predicable can't be set based on the variant, so left as no + (set_attr "predicable" "no, yes") (set_attr "length" "8")] ) diff --git a/gcc/config/arm/thumb2.md b/gcc/config/arm/thumb2.md index a5302f479f5091d08f4bf1a9ee9b0423692372af..0000000000000000000000000000000000000000 100644 --- a/gcc/config/arm/thumb2.md +++ b/gcc/config/arm/thumb2.md @@ -141,7 +141,7 @@ (define_insn "*thumb2_abssi2" eor%?\\t%0, %1, %1, asr #31\;sub%?\\t%0, %0, %1, asr #31" [(set_attr "conds" "clob,*") (set_attr "shift" "1") - ;; predicable can't be set based on the variant, so left as no + (set_attr "predicable" "no, yes") (set_attr "length" "10,8")] ) @@ -155,7 +155,7 @@ (define_insn "*thumb2_neg_abssi2" eor%?\\t%0, %1, %1, asr #31\;rsb%?\\t%0, %0, %1, asr #31" [(set_attr "conds" "clob,*") (set_attr "shift" "1") - ;; predicable can't be set based on the variant, so left as no + (set_attr "predicable" "no, yes") (set_attr "length" "10,8")] ) ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 2012-11-20 10:25 ` Kyrylo Tkachov @ 2012-11-20 10:32 ` Richard Earnshaw 2012-11-22 11:35 ` Kyrylo Tkachov 0 siblings, 1 reply; 6+ messages in thread From: Richard Earnshaw @ 2012-11-20 10:32 UTC (permalink / raw) To: Kyrylo Tkachov; +Cc: gcc-patches, Ramana Radhakrishnan On 20/11/12 10:25, Kyrylo Tkachov wrote: > Hi all, > This is the updated version of the patch. It defines the predicable > attribute for the equivalent abssi2 and neg_abssi2 patterns in thumb2.md. > The previous version (with the changes only to arm.md) has been okayed but > not applied and this patch supersedes it. > The previous version is: > http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01578.html > > No regressions for arm-none-eabi for qemu. > > Ok for trunk? > > Thanks, > Kyrill > > gcc/ChangeLog > > 2012-11-20 Kyrylo Tkachov <kyrylo.tkachov@arm.com> > > * config/arm/arm.md (*arm_abssi2): Define predicable attribute. > (*arm_neg_abssi2): Likewise. > * config/arm/thumb2.md (*thumb2_abssi2): Likewise. > (*thumb2_neg_abssi2): Likewise. > > > -----Original Message----- > From: Ramana Radhakrishnan > Sent: 19 November 2012 18:05 > To: Kyrylo Tkachov > Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw > Subject: Re: [PATCH][ARM] Define predicable attribute for arm_abssi2 and > arm_neg_abssi2 > > On 11/19/12 17:51, Kyrylo Tkachov wrote: >> Hi all, >> This patch updates the arm_abssi2 and arm_neg_abssi2 patterns in the ARM >> machine description. >> We define the predicable attribute based on the alternative. When the >> patterns were introduced it was not possible to do that. >> Now the second alternative in each of the patterns that supports > predication >> is defined as predicable. >> >> No regressions on arm-none-eabi with qemu. >> >> Ok for trunk? > > Ok, please apply. > > Ramana > OK. Thanks. R. ^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 2012-11-20 10:32 ` Richard Earnshaw @ 2012-11-22 11:35 ` Kyrylo Tkachov 0 siblings, 0 replies; 6+ messages in thread From: Kyrylo Tkachov @ 2012-11-22 11:35 UTC (permalink / raw) To: Richard Earnshaw; +Cc: gcc-patches, Ramana Radhakrishnan Thanks for the review. As I do not have write access, can someone please apply? http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01655.html Thanks, Kyrill -----Original Message----- From: Richard Earnshaw Sent: 20 November 2012 10:32 To: Kyrylo Tkachov Cc: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan Subject: Re: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 On 20/11/12 10:25, Kyrylo Tkachov wrote: > Hi all, > This is the updated version of the patch. It defines the predicable > attribute for the equivalent abssi2 and neg_abssi2 patterns in thumb2.md. > The previous version (with the changes only to arm.md) has been okayed but > not applied and this patch supersedes it. > The previous version is: > http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01578.html > > No regressions for arm-none-eabi for qemu. > > Ok for trunk? > > Thanks, > Kyrill > > gcc/ChangeLog > > 2012-11-20 Kyrylo Tkachov <kyrylo.tkachov@arm.com> > > * config/arm/arm.md (*arm_abssi2): Define predicable attribute. > (*arm_neg_abssi2): Likewise. > * config/arm/thumb2.md (*thumb2_abssi2): Likewise. > (*thumb2_neg_abssi2): Likewise. > > > -----Original Message----- > From: Ramana Radhakrishnan > Sent: 19 November 2012 18:05 > To: Kyrylo Tkachov > Cc: gcc-patches@gcc.gnu.org; Richard Earnshaw > Subject: Re: [PATCH][ARM] Define predicable attribute for arm_abssi2 and > arm_neg_abssi2 > > On 11/19/12 17:51, Kyrylo Tkachov wrote: >> Hi all, >> This patch updates the arm_abssi2 and arm_neg_abssi2 patterns in the ARM >> machine description. >> We define the predicable attribute based on the alternative. When the >> patterns were introduced it was not possible to do that. >> Now the second alternative in each of the patterns that supports > predication >> is defined as predicable. >> >> No regressions on arm-none-eabi with qemu. >> >> Ok for trunk? > > Ok, please apply. > > Ramana > OK. Thanks. R. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 2012-11-19 17:51 [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 Kyrylo Tkachov 2012-11-19 18:05 ` Ramana Radhakrishnan @ 2012-11-20 9:17 ` Richard Earnshaw 1 sibling, 0 replies; 6+ messages in thread From: Richard Earnshaw @ 2012-11-20 9:17 UTC (permalink / raw) To: Kyrylo Tkachov; +Cc: gcc-patches, Ramana Radhakrishnan On 19/11/12 17:51, Kyrylo Tkachov wrote: > Hi all, > This patch updates the arm_abssi2 and arm_neg_abssi2 patterns in the ARM > machine description. > We define the predicable attribute based on the alternative. When the > patterns were introduced it was not possible to do that. > Now the second alternative in each of the patterns that supports predication > is defined as predicable. > > No regressions on arm-none-eabi with qemu. > > Ok for trunk? > > Thanks, > Kyrill > > > gcc/ChangeLog > > 2012-11-19 Kyrylo Tkachov <kyrylo.tkachov@arm.com> > > * config/arm/arm.md (*arm_abssi2): Define predicable attribute. > (*arm_neg_abssi2): Define predicable attribute. > > There are equivalent patterns in thumb2.md as well. Could you fix those too, please. R. > abssi_predicable.txt > > > --- a/gcc/config/arm/arm.md > +++ b/gcc/config/arm/arm.md > @@ -4263,7 +4263,7 @@ (define_insn "*arm_abssi2" > eor%?\\t%0, %1, %1, asr #31\;sub%?\\t%0, %0, %1, asr #31" > [(set_attr "conds" "clob,*") > (set_attr "shift" "1") > - ;; predicable can't be set based on the variant, so left as no > + (set_attr "predicable" "no, yes") > (set_attr "length" "8")] > ) > > @@ -4291,7 +4291,7 @@ (define_insn "*arm_neg_abssi2" > eor%?\\t%0, %1, %1, asr #31\;rsb%?\\t%0, %0, %1, asr #31" > [(set_attr "conds" "clob,*") > (set_attr "shift" "1") > - ;; predicable can't be set based on the variant, so left as no > + (set_attr "predicable" "no, yes") > (set_attr "length" "8")] > ) > > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-11-22 11:35 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2012-11-19 17:51 [PATCH][ARM] Define predicable attribute for arm_abssi2 and arm_neg_abssi2 Kyrylo Tkachov 2012-11-19 18:05 ` Ramana Radhakrishnan 2012-11-20 10:25 ` Kyrylo Tkachov 2012-11-20 10:32 ` Richard Earnshaw 2012-11-22 11:35 ` Kyrylo Tkachov 2012-11-20 9:17 ` 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).