jina.nahias added inline comments.

================
Comment at: include/clang/Basic/BuiltinsX86.def:981
-TARGET_BUILTIN(__builtin_ia32_pbroadcastd512_gpr_mask, "V16iiV16iUs", "", 
"avx512f")
 TARGET_BUILTIN(__builtin_ia32_pbroadcastq512_mem_mask, "V8LLiLLiV8LLiUc", "", 
"avx512f")
 TARGET_BUILTIN(__builtin_ia32_loaddqusi512_mask, "V16iiC*V16iUs", "", 
"avx512f")
----------------
craig.topper wrote:
> I think you patch removed the only use of 
> __builtin_ia32_pbroadcastq512_mem_mask right? Does your change work properly 
> in 32-bit mode?
yes for both of the questions, i have deleted 
__builtin_ia32_pbroadcastq512_mem_mask  you can see it in the new update.


https://reviews.llvm.org/D37668



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to