[PATCH] D60552: [X86] Enable intrinsics of AVX512_BF16, which are supported for BFLOAT16 in Cooper Lake

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: lib/Basic/Targets/X86.cpp:667
     if (Name == "avx512bw" && !Enabled)
       Features["avx512vbmi"] = Features["avx512vbmi2"] =
       Features["avx512bitalg"] = false;
----------------
craig.topper wrote:
> Need to also disable avx512fp16 when we disable "avx512vbmi" and "avx512vbmi2". 
> 
> Need a new if to disable "avx512fp16" when "if (Name == "avx512vl" && !Enabled)"
That should have said avx512bf16. 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60552/new/

https://reviews.llvm.org/D60552