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

[Valentin Churavy via Phabricator via llvm-commits]

vchuravy updated this revision to Diff 82848.
vchuravy added a comment.

groups the model id for skylake and checks for AVX512 to differentiate between SKL and SKX


https://reviews.llvm.org/D28221

Files:
  lib/Support/Host.cpp


Index: lib/Support/Host.cpp
===================================================================
--- lib/Support/Host.cpp
+++ lib/Support/Host.cpp
@@ -475,12 +475,13 @@
 
     // Skylake:
     case 0x4e:
-      *Type = INTEL_COREI7; // "skylake-avx512"
-      *Subtype = INTEL_COREI7_SKYLAKE_AVX512;
-      break;
     case 0x5e:
-      *Type = INTEL_COREI7; // "skylake"
-      *Subtype = INTEL_COREI7_SKYLAKE;
+      *Type = INTEL_COREI7;
+      if (Features & (1 << FEATURE_AVX512)) {
+        *Subtype = INTEL_COREI7_SKYLAKE_AVX512; // "skylake-avx512"
+      } else {
+        *Subtype = INTEL_COREI7_SKYLAKE; // "skylake"
+      }
       break;
 
     case 0x1c: // Most 45 nm Intel Atom processors


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28221.82848.patch
Type: text/x-patch
Size: 704 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170103/565f24e6/attachment.bin>