* [PATCh] Improve sse_mov{hl,lh}ps
@ 2016-05-04 19:37 Jakub Jelinek
2016-05-06 11:46 ` Kirill Yukhin
0 siblings, 1 reply; 2+ messages in thread
From: Jakub Jelinek @ 2016-05-04 19:37 UTC (permalink / raw)
To: Uros Bizjak, Kirill Yukhin; +Cc: gcc-patches
Hi!
Another pair of define_insns where all the VEX insns have EVEX variant
in AVX512VL.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2016-05-04 Jakub Jelinek <jakub@redhat.com>
* config/i386/sse.md (sse_movhlps, sse_movlhps): Use
v instead of x in vex or maybe_vex alternatives, use
maybe_evex instead of vex in prefix.
--- gcc/config/i386/sse.md.jj 2016-05-04 14:36:08.000000000 +0200
+++ gcc/config/i386/sse.md 2016-05-04 15:16:44.180894303 +0200
@@ -5744,11 +5744,11 @@ (define_expand "sse_movhlps_exp"
})
(define_insn "sse_movhlps"
- [(set (match_operand:V4SF 0 "nonimmediate_operand" "=x,x,x,x,m")
+ [(set (match_operand:V4SF 0 "nonimmediate_operand" "=x,v,x,v,m")
(vec_select:V4SF
(vec_concat:V8SF
- (match_operand:V4SF 1 "nonimmediate_operand" " 0,x,0,x,0")
- (match_operand:V4SF 2 "nonimmediate_operand" " x,x,o,o,x"))
+ (match_operand:V4SF 1 "nonimmediate_operand" " 0,v,0,v,0")
+ (match_operand:V4SF 2 "nonimmediate_operand" " x,v,o,o,v"))
(parallel [(const_int 6)
(const_int 7)
(const_int 2)
@@ -5762,7 +5762,7 @@ (define_insn "sse_movhlps"
%vmovhps\t{%2, %0|%q0, %2}"
[(set_attr "isa" "noavx,avx,noavx,avx,*")
(set_attr "type" "ssemov")
- (set_attr "prefix" "orig,vex,orig,vex,maybe_vex")
+ (set_attr "prefix" "orig,maybe_evex,orig,maybe_evex,maybe_vex")
(set_attr "mode" "V4SF,V4SF,V2SF,V2SF,V2SF")])
(define_expand "sse_movlhps_exp"
@@ -5789,11 +5789,11 @@ (define_expand "sse_movlhps_exp"
})
(define_insn "sse_movlhps"
- [(set (match_operand:V4SF 0 "nonimmediate_operand" "=x,x,x,x,o")
+ [(set (match_operand:V4SF 0 "nonimmediate_operand" "=x,v,x,v,o")
(vec_select:V4SF
(vec_concat:V8SF
- (match_operand:V4SF 1 "nonimmediate_operand" " 0,x,0,x,0")
- (match_operand:V4SF 2 "nonimmediate_operand" " x,x,m,m,x"))
+ (match_operand:V4SF 1 "nonimmediate_operand" " 0,v,0,v,0")
+ (match_operand:V4SF 2 "nonimmediate_operand" " x,v,m,v,v"))
(parallel [(const_int 0)
(const_int 1)
(const_int 4)
@@ -5807,7 +5807,7 @@ (define_insn "sse_movlhps"
%vmovlps\t{%2, %H0|%H0, %2}"
[(set_attr "isa" "noavx,avx,noavx,avx,*")
(set_attr "type" "ssemov")
- (set_attr "prefix" "orig,vex,orig,vex,maybe_vex")
+ (set_attr "prefix" "orig,maybe_evex,orig,maybe_evex,maybe_vex")
(set_attr "mode" "V4SF,V4SF,V2SF,V2SF,V2SF")])
(define_insn "<mask_codefor>avx512f_unpckhps512<mask_name>"
Jakub
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCh] Improve sse_mov{hl,lh}ps
2016-05-04 19:37 [PATCh] Improve sse_mov{hl,lh}ps Jakub Jelinek
@ 2016-05-06 11:46 ` Kirill Yukhin
0 siblings, 0 replies; 2+ messages in thread
From: Kirill Yukhin @ 2016-05-06 11:46 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: Uros Bizjak, gcc-patches
On 04 May 21:37, Jakub Jelinek wrote:
> Hi!
>
> Another pair of define_insns where all the VEX insns have EVEX variant
> in AVX512VL.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
OK for trunk.
--
Thanks, K
>
> 2016-05-04 Jakub Jelinek <jakub@redhat.com>
>
> * config/i386/sse.md (sse_movhlps, sse_movlhps): Use
> v instead of x in vex or maybe_vex alternatives, use
> maybe_evex instead of vex in prefix.
>
> --- gcc/config/i386/sse.md.jj 2016-05-04 14:36:08.000000000 +0200
> +++ gcc/config/i386/sse.md 2016-05-04 15:16:44.180894303 +0200
> @@ -5744,11 +5744,11 @@ (define_expand "sse_movhlps_exp"
> })
>
> (define_insn "sse_movhlps"
> - [(set (match_operand:V4SF 0 "nonimmediate_operand" "=x,x,x,x,m")
> + [(set (match_operand:V4SF 0 "nonimmediate_operand" "=x,v,x,v,m")
> (vec_select:V4SF
> (vec_concat:V8SF
> - (match_operand:V4SF 1 "nonimmediate_operand" " 0,x,0,x,0")
> - (match_operand:V4SF 2 "nonimmediate_operand" " x,x,o,o,x"))
> + (match_operand:V4SF 1 "nonimmediate_operand" " 0,v,0,v,0")
> + (match_operand:V4SF 2 "nonimmediate_operand" " x,v,o,o,v"))
> (parallel [(const_int 6)
> (const_int 7)
> (const_int 2)
> @@ -5762,7 +5762,7 @@ (define_insn "sse_movhlps"
> %vmovhps\t{%2, %0|%q0, %2}"
> [(set_attr "isa" "noavx,avx,noavx,avx,*")
> (set_attr "type" "ssemov")
> - (set_attr "prefix" "orig,vex,orig,vex,maybe_vex")
> + (set_attr "prefix" "orig,maybe_evex,orig,maybe_evex,maybe_vex")
> (set_attr "mode" "V4SF,V4SF,V2SF,V2SF,V2SF")])
>
> (define_expand "sse_movlhps_exp"
> @@ -5789,11 +5789,11 @@ (define_expand "sse_movlhps_exp"
> })
>
> (define_insn "sse_movlhps"
> - [(set (match_operand:V4SF 0 "nonimmediate_operand" "=x,x,x,x,o")
> + [(set (match_operand:V4SF 0 "nonimmediate_operand" "=x,v,x,v,o")
> (vec_select:V4SF
> (vec_concat:V8SF
> - (match_operand:V4SF 1 "nonimmediate_operand" " 0,x,0,x,0")
> - (match_operand:V4SF 2 "nonimmediate_operand" " x,x,m,m,x"))
> + (match_operand:V4SF 1 "nonimmediate_operand" " 0,v,0,v,0")
> + (match_operand:V4SF 2 "nonimmediate_operand" " x,v,m,v,v"))
> (parallel [(const_int 0)
> (const_int 1)
> (const_int 4)
> @@ -5807,7 +5807,7 @@ (define_insn "sse_movlhps"
> %vmovlps\t{%2, %H0|%H0, %2}"
> [(set_attr "isa" "noavx,avx,noavx,avx,*")
> (set_attr "type" "ssemov")
> - (set_attr "prefix" "orig,vex,orig,vex,maybe_vex")
> + (set_attr "prefix" "orig,maybe_evex,orig,maybe_evex,maybe_vex")
> (set_attr "mode" "V4SF,V4SF,V2SF,V2SF,V2SF")])
>
> (define_insn "<mask_codefor>avx512f_unpckhps512<mask_name>"
>
> Jakub
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-05-06 11:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-04 19:37 [PATCh] Improve sse_mov{hl,lh}ps Jakub Jelinek
2016-05-06 11:46 ` Kirill Yukhin
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).