Re: [PATCH] Fix ix86_expand_vector_set (PR target/70421)

2016-03-30 Thread Jakub Jelinek
On Wed, Mar 30, 2016 at 04:53:48PM +0300, Kirill Yukhin wrote: > I think that order of arguments in built-in might be changed easily. > This doesn't affect intrinsics at all, because of that. > > I can hardly recall, but my bet is that this order was dictated by: > ix86_expand_sse_movcc where orde

Re: [PATCH] Fix ix86_expand_vector_set (PR target/70421)

2016-03-30 Thread Kirill Yukhin
On 29 Mar 19:49, Jakub Jelinek wrote: > On Tue, Mar 29, 2016 at 11:44:15AM -0600, Jeff Law wrote: > > On 03/29/2016 11:05 AM, Jakub Jelinek wrote: > > >Hi! > > > > > >The various blendm expanders look like: > > >(define_insn "_blendm" > > > [(set (match_operand:V48_AVX512VL 0 "register_operand" "

Re: [PATCH] Fix ix86_expand_vector_set (PR target/70421)

2016-03-29 Thread Jakub Jelinek
On Tue, Mar 29, 2016 at 11:44:15AM -0600, Jeff Law wrote: > On 03/29/2016 11:05 AM, Jakub Jelinek wrote: > >Hi! > > > >The various blendm expanders look like: > >(define_insn "_blendm" > > [(set (match_operand:V48_AVX512VL 0 "register_operand" "=v") > > (vec_merge:V48_AVX512VL > >

Re: [PATCH] Fix ix86_expand_vector_set (PR target/70421)

2016-03-29 Thread Jeff Law
On 03/29/2016 11:05 AM, Jakub Jelinek wrote: Hi! The various blendm expanders look like: (define_insn "_blendm" [(set (match_operand:V48_AVX512VL 0 "register_operand" "=v") (vec_merge:V48_AVX512VL (match_operand:V48_AVX512VL 2 "nonimmediate_operand" "vm") (match

[PATCH] Fix ix86_expand_vector_set (PR target/70421)

2016-03-29 Thread Jakub Jelinek
Hi! The various blendm expanders look like: (define_insn "_blendm" [(set (match_operand:V48_AVX512VL 0 "register_operand" "=v") (vec_merge:V48_AVX512VL (match_operand:V48_AVX512VL 2 "nonimmediate_operand" "vm") (match_operand:V48_AVX512VL 1 "register_operand" "v")