* [PATCH] nvptx: Support 16-bit shifts and extendqihi2.
@ 2020-07-11 18:44 Roger Sayle
2020-07-28 13:15 ` Tom de Vries
0 siblings, 1 reply; 2+ messages in thread
From: Roger Sayle @ 2020-07-11 18:44 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 457 bytes --]
The following patch adds support for 16-bits shifts and for sign extension
from 8 bits to 16 bits.
This patch has been tested on nvptx-none with no new regressions.
Ok for mainline?
2020-07-11 Roger Sayle <roger@nextmovesoftware.com>
gcc/ChangeLog
* config/nvptx/nvptx.md (extendqihi2): New instruction.
(ashl<mode>3, ashr<mode>3, lshr<mode>3): Support HImode.
Thanks in advance,
Roger
--
Roger Sayle
NextMove Software
Cambridge, UK
[-- Attachment #2: patchg2.txt --]
[-- Type: text/plain, Size: 2120 bytes --]
diff --git a/gcc/config/nvptx/nvptx.md b/gcc/config/nvptx/nvptx.md
index 6545b81..0538e83 100644
--- a/gcc/config/nvptx/nvptx.md
+++ b/gcc/config/nvptx/nvptx.md
@@ -319,6 +319,13 @@
%.\\tld%A1%u1\\t%0, %1;"
[(set_attr "subregs_ok" "true")])
+(define_insn "extendqihi2"
+ [(set (match_operand:HI 0 "nvptx_register_operand" "=R")
+ (sign_extend:HI (match_operand:QI 1 "nvptx_register_operand" "R")))]
+ ""
+ "%.\\tcvt.s16.s8\\t%0, %1;"
+ [(set_attr "subregs_ok" "true")])
+
(define_insn "extend<mode>si2"
[(set (match_operand:SI 0 "nvptx_register_operand" "=R,R")
(sign_extend:SI (match_operand:QHIM 1 "nvptx_nonimmediate_operand" "R,m")))]
@@ -556,23 +563,23 @@
;; Shifts
(define_insn "ashl<mode>3"
- [(set (match_operand:SDIM 0 "nvptx_register_operand" "=R")
- (ashift:SDIM (match_operand:SDIM 1 "nvptx_register_operand" "R")
- (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
+ [(set (match_operand:HSDIM 0 "nvptx_register_operand" "=R")
+ (ashift:HSDIM (match_operand:HSDIM 1 "nvptx_register_operand" "R")
+ (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
""
"%.\\tshl.b%T0\\t%0, %1, %2;")
(define_insn "ashr<mode>3"
- [(set (match_operand:SDIM 0 "nvptx_register_operand" "=R")
- (ashiftrt:SDIM (match_operand:SDIM 1 "nvptx_register_operand" "R")
- (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
+ [(set (match_operand:HSDIM 0 "nvptx_register_operand" "=R")
+ (ashiftrt:HSDIM (match_operand:HSDIM 1 "nvptx_register_operand" "R")
+ (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
""
"%.\\tshr.s%T0\\t%0, %1, %2;")
(define_insn "lshr<mode>3"
- [(set (match_operand:SDIM 0 "nvptx_register_operand" "=R")
- (lshiftrt:SDIM (match_operand:SDIM 1 "nvptx_register_operand" "R")
- (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
+ [(set (match_operand:HSDIM 0 "nvptx_register_operand" "=R")
+ (lshiftrt:HSDIM (match_operand:HSDIM 1 "nvptx_register_operand" "R")
+ (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
""
"%.\\tshr.u%T0\\t%0, %1, %2;")
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] nvptx: Support 16-bit shifts and extendqihi2.
2020-07-11 18:44 [PATCH] nvptx: Support 16-bit shifts and extendqihi2 Roger Sayle
@ 2020-07-28 13:15 ` Tom de Vries
0 siblings, 0 replies; 2+ messages in thread
From: Tom de Vries @ 2020-07-28 13:15 UTC (permalink / raw)
To: Roger Sayle, gcc-patches
[-- Attachment #1: Type: text/plain, Size: 509 bytes --]
On 7/11/20 8:44 PM, Roger Sayle wrote:
> The following patch adds support for 16-bits shifts and for sign extension
> from 8 bits to 16 bits.
>
> This patch has been tested on nvptx-none with no new regressions.
> Ok for mainline?
>
> 2020-07-11 Roger Sayle <roger@nextmovesoftware.com>
>
> gcc/ChangeLog
> * config/nvptx/nvptx.md (extendqihi2): New instruction.
> (ashl<mode>3, ashr<mode>3, lshr<mode>3): Support HImode.
>
>
Pushed as attached, with test-case added.
Thanks,
- Tom
[-- Attachment #2: 0001-nvptx-Support-16-bit-shifts-and-extendqihi2.patch --]
[-- Type: text/x-patch, Size: 4067 bytes --]
nvptx: Support 16-bit shifts and extendqihi2
Add support for 16-bits shifts and for sign extension from 8 bits to
16 bits.
This patch has been tested on nvptx-none with no new regressions.
2020-07-11 Roger Sayle <roger@nextmovesoftware.com>
Tom de Vries <tdevries@suse.de>
gcc/ChangeLog:
* config/nvptx/nvptx.md (extendqihi2): New instruction.
(ashl<mode>3, ashr<mode>3, lshr<mode>3): Support HImode.
gcc/testsuite/ChangeLog:
* gcc.target/nvptx/cvt.c: New test.
* gcc.target/nvptx/shift16.c: New test.
---
gcc/config/nvptx/nvptx.md | 25 ++++++++++++++++---------
gcc/testsuite/gcc.target/nvptx/cvt.c | 13 +++++++++++++
gcc/testsuite/gcc.target/nvptx/shift16.c | 30 ++++++++++++++++++++++++++++++
3 files changed, 59 insertions(+), 9 deletions(-)
diff --git a/gcc/config/nvptx/nvptx.md b/gcc/config/nvptx/nvptx.md
index 6545b81f948..0538e834a4c 100644
--- a/gcc/config/nvptx/nvptx.md
+++ b/gcc/config/nvptx/nvptx.md
@@ -319,6 +319,13 @@
%.\\tld%A1%u1\\t%0, %1;"
[(set_attr "subregs_ok" "true")])
+(define_insn "extendqihi2"
+ [(set (match_operand:HI 0 "nvptx_register_operand" "=R")
+ (sign_extend:HI (match_operand:QI 1 "nvptx_register_operand" "R")))]
+ ""
+ "%.\\tcvt.s16.s8\\t%0, %1;"
+ [(set_attr "subregs_ok" "true")])
+
(define_insn "extend<mode>si2"
[(set (match_operand:SI 0 "nvptx_register_operand" "=R,R")
(sign_extend:SI (match_operand:QHIM 1 "nvptx_nonimmediate_operand" "R,m")))]
@@ -556,23 +563,23 @@
;; Shifts
(define_insn "ashl<mode>3"
- [(set (match_operand:SDIM 0 "nvptx_register_operand" "=R")
- (ashift:SDIM (match_operand:SDIM 1 "nvptx_register_operand" "R")
- (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
+ [(set (match_operand:HSDIM 0 "nvptx_register_operand" "=R")
+ (ashift:HSDIM (match_operand:HSDIM 1 "nvptx_register_operand" "R")
+ (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
""
"%.\\tshl.b%T0\\t%0, %1, %2;")
(define_insn "ashr<mode>3"
- [(set (match_operand:SDIM 0 "nvptx_register_operand" "=R")
- (ashiftrt:SDIM (match_operand:SDIM 1 "nvptx_register_operand" "R")
- (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
+ [(set (match_operand:HSDIM 0 "nvptx_register_operand" "=R")
+ (ashiftrt:HSDIM (match_operand:HSDIM 1 "nvptx_register_operand" "R")
+ (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
""
"%.\\tshr.s%T0\\t%0, %1, %2;")
(define_insn "lshr<mode>3"
- [(set (match_operand:SDIM 0 "nvptx_register_operand" "=R")
- (lshiftrt:SDIM (match_operand:SDIM 1 "nvptx_register_operand" "R")
- (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
+ [(set (match_operand:HSDIM 0 "nvptx_register_operand" "=R")
+ (lshiftrt:HSDIM (match_operand:HSDIM 1 "nvptx_register_operand" "R")
+ (match_operand:SI 2 "nvptx_nonmemory_operand" "Ri")))]
""
"%.\\tshr.u%T0\\t%0, %1, %2;")
diff --git a/gcc/testsuite/gcc.target/nvptx/cvt.c b/gcc/testsuite/gcc.target/nvptx/cvt.c
new file mode 100644
index 00000000000..279ec16b14c
--- /dev/null
+++ b/gcc/testsuite/gcc.target/nvptx/cvt.c
@@ -0,0 +1,13 @@
+/* { dg-do assemble } */
+/* { dg-options "-O2 -save-temps" } */
+
+signed short s;
+signed char c;
+
+void
+foo (void)
+{
+ s = c;
+}
+
+/* { dg-final { scan-assembler "(?n)cvt\\.s16\\.s8.*%r" } } */
diff --git a/gcc/testsuite/gcc.target/nvptx/shift16.c b/gcc/testsuite/gcc.target/nvptx/shift16.c
new file mode 100644
index 00000000000..185aa62fa9a
--- /dev/null
+++ b/gcc/testsuite/gcc.target/nvptx/shift16.c
@@ -0,0 +1,30 @@
+/* { dg-do assemble } */
+/* { dg-options "-O2 -save-temps" } */
+
+void
+foo (unsigned short u)
+{
+ volatile unsigned short u2 = u << 5;
+}
+
+void
+foo2 (short s)
+{
+ volatile unsigned short s2 = s << 5;
+}
+
+void
+foo3 (unsigned short u)
+{
+ volatile unsigned short u2 = u >> 5;
+}
+
+void
+foo4 (signed short s)
+{
+ volatile signed short s2 = s >> 5;
+}
+
+/* { dg-final { scan-assembler-times "(?n)shl\\.b16.*%r" 2 } } */
+/* { dg-final { scan-assembler "(?n)shr\\.u16.*%r" } } */
+/* { dg-final { scan-assembler "(?n)shr\\.s16.*%r" } } */
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-07-28 13:15 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-11 18:44 [PATCH] nvptx: Support 16-bit shifts and extendqihi2 Roger Sayle
2020-07-28 13:15 ` Tom de Vries
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).