On 11/05/2013 11:05 PM, Kirill Yukhin wrote: > Hello, > Small correction. > > On 01 Nov 16:19, Kirill Yukhin wrote: >> +(define_insn "avx512f_store<mode>_mask" >> + [(set (match_operand:VI48F_512 0 "memory_operand" "=m") >> + (vec_merge:VI48F_512 >> + (match_operand:VI48F_512 1 "register_operand" "v") >> + (match_dup 0) >> + (match_operand:<avx512fmaskmode> 2 "register_operand" "k")))] >> + "TARGET_AVX512F" >> +{ >> + switch (<sseinsnmode>mode) > Need to be MODE_<sseinsnmode>. Same for load. > > Is it ok with that change?
Yes, of course that's what I meant. r~