[PATCH] D28221: Add check for AVX512 support before assuming skylake processor is SKX.

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added inline comments.


================
Comment at: lib/Support/Host.cpp:485
+    case 0x55:
+      *Type = INTEL_COREI7; // "skylake-avx512"
+      // Check that we really have AVX512
----------------
Should the:

  // "skylake-avx512"

comment be moved down to where the Subtype is set?



https://reviews.llvm.org/D28221