* [PATCH 16/20] aarch64: Relax aarch64_<sur><addsub>hn2<mode> RTL pattern
@ 2021-04-28 14:53 Jonathan Wright
2021-04-28 15:57 ` Richard Sandiford
0 siblings, 1 reply; 2+ messages in thread
From: Jonathan Wright @ 2021-04-28 14:53 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 788 bytes --]
Hi,
As subject, this patch implements the v[r]addhn2 and v[r]subhn2 Neon
intrinsic RTL patterns using a vec_concat of a register_operand and an
ADDSUBHN unspec - instead of just an ADDSUBHN2 unspec. This more
relaxed pattern allows for more aggressive combinations and ultimately
better code generation.
Regression tested and bootstrapped on aarch64-none-linux-gnu and
aarch64_be-none-elf - no issues.
Ok for master?
Thanks,
Jonathan
---
gcc/ChangeLog:
2021-03-03 Jonathan Wright <jonathan.wright@arm.com>
* config/aarch64/aarch64-simd.md (aarch64_<sur><addsub>hn2<mode>):
Implement as an expand emitting a big/little endian
instruction pattern.
(aarch64_<sur><addsub>hn2<mode>_insn_le): Define.
(aarch64_<sur><addsub>hn2<mode>_insn_be): Define.
[-- Attachment #2: rb14250.patch --]
[-- Type: application/octet-stream, Size: 2303 bytes --]
diff --git a/gcc/config/aarch64/aarch64-simd.md b/gcc/config/aarch64/aarch64-simd.md
index fa4972e769ee79dfd369223e867ed18beb6dfb2c..4da5711eaedeb77da1bbeab49af637a7fafea9c2 100644
--- a/gcc/config/aarch64/aarch64-simd.md
+++ b/gcc/config/aarch64/aarch64-simd.md
@@ -4652,17 +4652,48 @@
[(set_attr "type" "neon_<addsub>_halve_narrow_q")]
)
-(define_insn "aarch64_<sur><addsub>hn2<mode>"
+(define_insn "aarch64_<sur><addsub>hn2<mode>_insn_le"
[(set (match_operand:<VNARROWQ2> 0 "register_operand" "=w")
- (unspec:<VNARROWQ2> [(match_operand:<VNARROWQ> 1 "register_operand" "0")
- (match_operand:VQN 2 "register_operand" "w")
- (match_operand:VQN 3 "register_operand" "w")]
- ADDSUBHN2))]
- "TARGET_SIMD"
+ (vec_concat:<VNARROWQ2>
+ (match_operand:<VNARROWQ> 1 "register_operand" "0")
+ (unspec:<VNARROWQ> [(match_operand:VQN 2 "register_operand" "w")
+ (match_operand:VQN 3 "register_operand" "w")]
+ ADDSUBHN)))]
+ "TARGET_SIMD && !BYTES_BIG_ENDIAN"
"<sur><addsub>hn2\\t%0.<V2ntype>, %2.<Vtype>, %3.<Vtype>"
[(set_attr "type" "neon_<addsub>_halve_narrow_q")]
)
+(define_insn "aarch64_<sur><addsub>hn2<mode>_insn_be"
+ [(set (match_operand:<VNARROWQ2> 0 "register_operand" "=w")
+ (vec_concat:<VNARROWQ2>
+ (unspec:<VNARROWQ> [(match_operand:VQN 2 "register_operand" "w")
+ (match_operand:VQN 3 "register_operand" "w")]
+ ADDSUBHN)
+ (match_operand:<VNARROWQ> 1 "register_operand" "0")))]
+ "TARGET_SIMD && BYTES_BIG_ENDIAN"
+ "<sur><addsub>hn2\\t%0.<V2ntype>, %2.<Vtype>, %3.<Vtype>"
+ [(set_attr "type" "neon_<addsub>_halve_narrow_q")]
+)
+
+(define_expand "aarch64_<sur><addsub>hn2<mode>"
+ [(match_operand:<VNARROWQ2> 0 "register_operand")
+ (unspec:<VNARROWQ2> [(match_operand:<VNARROWQ> 1 "register_operand")
+ (match_operand:VQN 2 "register_operand")
+ (match_operand:VQN 3 "register_operand")]
+ ADDSUBHN2)]
+ "TARGET_SIMD"
+ {
+ if (BYTES_BIG_ENDIAN)
+ emit_insn (gen_aarch64_<sur><addsub>hn2<mode>_insn_be (operands[0],
+ operands[1], operands[2], operands[3]));
+ else
+ emit_insn (gen_aarch64_<sur><addsub>hn2<mode>_insn_le (operands[0],
+ operands[1], operands[2], operands[3]));
+ DONE;
+ }
+)
+
;; pmul.
(define_insn "aarch64_pmul<mode>"
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 16/20] aarch64: Relax aarch64_<sur><addsub>hn2<mode> RTL pattern
2021-04-28 14:53 [PATCH 16/20] aarch64: Relax aarch64_<sur><addsub>hn2<mode> RTL pattern Jonathan Wright
@ 2021-04-28 15:57 ` Richard Sandiford
0 siblings, 0 replies; 2+ messages in thread
From: Richard Sandiford @ 2021-04-28 15:57 UTC (permalink / raw)
To: Jonathan Wright via Gcc-patches
Jonathan Wright via Gcc-patches <gcc-patches@gcc.gnu.org> writes:
> Hi,
>
> As subject, this patch implements the v[r]addhn2 and v[r]subhn2 Neon
> intrinsic RTL patterns using a vec_concat of a register_operand and an
> ADDSUBHN unspec - instead of just an ADDSUBHN2 unspec. This more
> relaxed pattern allows for more aggressive combinations and ultimately
> better code generation.
With this and patch 17, it would be good to have some tests that
show (and defend) the improvement.
Thanks,
Richard
>
> Regression tested and bootstrapped on aarch64-none-linux-gnu and
> aarch64_be-none-elf - no issues.
>
> Ok for master?
>
> Thanks,
> Jonathan
>
> ---
>
> gcc/ChangeLog:
>
> 2021-03-03 Jonathan Wright <jonathan.wright@arm.com>
>
> * config/aarch64/aarch64-simd.md (aarch64_<sur><addsub>hn2<mode>):
> Implement as an expand emitting a big/little endian
> instruction pattern.
> (aarch64_<sur><addsub>hn2<mode>_insn_le): Define.
> (aarch64_<sur><addsub>hn2<mode>_insn_be): Define.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-04-28 15:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-28 14:53 [PATCH 16/20] aarch64: Relax aarch64_<sur><addsub>hn2<mode> RTL pattern Jonathan Wright
2021-04-28 15:57 ` 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).