[PATCH] D56460: [X86] Disable DomainReassignment pass when AVX512BW is disabled to avoid injecting VK32/VK64 references into the MachineIR

Wed Jan 9 07:09:34 PST 2019

spatel added a comment.

I still don't know much about AVX512, but code change is trivially safer. Upload patch with context for better future searchability?

================
Comment at: lib/Target/X86/X86DomainReassignment.cpp:735
   // AVX512.
-  if (!STI->hasAVX512())
+  // TODO: We're also bailing of AVX512BW isn't supported since we use VK32 and
+  // VK64 for GR32/GR64, but those aren't legal classes on KNL. If the register
----------------
of -> if

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

https://reviews.llvm.org/D56460