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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
