[PATCH] D33170: [X86] Adding avx512_vpopcntdq feature set and its intrinsics

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a reviewer: RKSimon.
RKSimon added inline comments.


================
Comment at: include/clang/Basic/BuiltinsX86.def:1105
+TARGET_BUILTIN(__builtin_ia32_vpopcntd_512_mask, "V16iV16iV16iUs", "", "avx512_vpopcntdq")
+TARGET_BUILTIN(__builtin_ia32_vpopcntq_512_mask, "V8LLiV8LLiV8LLiUc", "", "avx512_vpopcntdq")
+
----------------
Would we be better off directly emitting Intrinsic::ctpop inside CGBuiltin.cpp?

AFAICT these builtins have no additional abilities compared to ctpop (other than masking which we can already handle).


Repository:
  rL LLVM

https://reviews.llvm.org/D33170