public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH][ARM] Add mode to probe_stack set operands
@ 2016-05-03 11:27 Kyrill Tkachov
  2016-05-09  9:21 ` Richard Earnshaw (lists)
  0 siblings, 1 reply; 2+ messages in thread
From: Kyrill Tkachov @ 2016-05-03 11:27 UTC (permalink / raw)
  To: GCC Patches; +Cc: Ramana Radhakrishnan, Richard Earnshaw

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

Hi all,

When building the arm backend genrecog complains that the probe_stack set expression
doesn't specify any modes.  This patch adds the SI mode annotation and fixes the warning

Bootstrapped and tested on arm-none-linux-gnueabihf.

Ok for trunk?

Thanks,
Kyrill

2016-05-03  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

     * config/arm/arm.md (probe_stack): Add modes to set source
     and destination.

[-- Attachment #2: arm-stack-probe.patch --]
[-- Type: text/x-patch, Size: 573 bytes --]

diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md
index 68de70f05ce29f40849297e160d890f033c34487..0d491f7ea41e4fb5fb58bbb3047294abda541a73 100644
--- a/gcc/config/arm/arm.md
+++ b/gcc/config/arm/arm.md
@@ -8091,8 +8091,8 @@ (define_insn "blockage"
 )
 
 (define_insn "probe_stack"
-  [(set (match_operand 0 "memory_operand" "=m")
-        (unspec [(const_int 0)] UNSPEC_PROBE_STACK))]
+  [(set (match_operand:SI 0 "memory_operand" "=m")
+        (unspec:SI [(const_int 0)] UNSPEC_PROBE_STACK))]
   "TARGET_32BIT"
   "str%?\\tr0, %0"
   [(set_attr "type" "store1")

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

* Re: [PATCH][ARM] Add mode to probe_stack set operands
  2016-05-03 11:27 [PATCH][ARM] Add mode to probe_stack set operands Kyrill Tkachov
@ 2016-05-09  9:21 ` Richard Earnshaw (lists)
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Earnshaw (lists) @ 2016-05-09  9:21 UTC (permalink / raw)
  To: Kyrill Tkachov, GCC Patches; +Cc: Ramana Radhakrishnan

On 03/05/16 12:27, Kyrill Tkachov wrote:
> Hi all,
> 
> When building the arm backend genrecog complains that the probe_stack
> set expression
> doesn't specify any modes.  This patch adds the SI mode annotation and
> fixes the warning
> 
> Bootstrapped and tested on arm-none-linux-gnueabihf.
> 
> Ok for trunk?
> 
> Thanks,
> Kyrill
> 
> 2016-05-03  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
> 
>     * config/arm/arm.md (probe_stack): Add modes to set source
>     and destination.
> 

OK.

R.

> arm-stack-probe.patch
> 
> 
> diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md
> index 68de70f05ce29f40849297e160d890f033c34487..0d491f7ea41e4fb5fb58bbb3047294abda541a73 100644
> --- a/gcc/config/arm/arm.md
> +++ b/gcc/config/arm/arm.md
> @@ -8091,8 +8091,8 @@ (define_insn "blockage"
>  )
>  
>  (define_insn "probe_stack"
> -  [(set (match_operand 0 "memory_operand" "=m")
> -        (unspec [(const_int 0)] UNSPEC_PROBE_STACK))]
> +  [(set (match_operand:SI 0 "memory_operand" "=m")
> +        (unspec:SI [(const_int 0)] UNSPEC_PROBE_STACK))]
>    "TARGET_32BIT"
>    "str%?\\tr0, %0"
>    [(set_attr "type" "store1")
> 

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

end of thread, other threads:[~2016-05-09  9:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-03 11:27 [PATCH][ARM] Add mode to probe_stack set operands Kyrill Tkachov
2016-05-09  9:21 ` Richard Earnshaw (lists)

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