On 08/14/2013 12:23 AM, Kirill Yukhin wrote:
> +  ;; For AVX512F mask support
> +  UNSPEC_KIOR
> +  UNSPEC_KXOR
> +  UNSPEC_KAND
> +  UNSPEC_KANDN

I thought we determined that you didn't need these,
that "*Yk" as a constraint was sufficient.

> +(define_insn "kandn<mode>"
> +(define_insn "kand<mode>"
> +(define_insn "kior<mode>"
> +(define_insn "kxor<mode>"

Because otherwise there's nothing different between these...

> +(define_insn "kxnor<mode>"
> +(define_insn "kortestzhi"
> +(define_insn "kortestchi"
> +(define_insn "kunpckhi"
>  (define_insn "*one_cmpl<mode>2_1"
> +(define_insn "*one_cmplhi2_1"
>  (define_insn "*one_cmplqi2_1"

... and these.



r~

Reply via email to