[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 83022.
vchuravy added a comment.

Add model id 0x55 as SKX and consolidate 0x4e & 0x5e into SKL

This follows the Linux kernel [1].

[1] https://github.com/torvalds/linux/blob/08328814256d888634ff15ba8fb67e2ae4340b64/arch/x86/include/asm/intel-family.h#L43-L45


https://reviews.llvm.org/D28221

Files:
  lib/Support/Host.cpp


Index: lib/Support/Host.cpp
===================================================================
--- lib/Support/Host.cpp
+++ lib/Support/Host.cpp
@@ -475,13 +475,14 @@
 
     // Skylake:
     case 0x4e:
-      *Type = INTEL_COREI7; // "skylake-avx512"
-      *Subtype = INTEL_COREI7_SKYLAKE_AVX512;
-      break;
     case 0x5e:
       *Type = INTEL_COREI7; // "skylake"
       *Subtype = INTEL_COREI7_SKYLAKE;
       break;
+    case 0x55:
+      *Type = INTEL_COREI7; // "skylake-avx512"
+      *Subtype = INTEL_COREI7_SKYLAKE_AVX512;
+      break;
 
     case 0x1c: // Most 45 nm Intel Atom processors
     case 0x26: // 45 nm Atom Lincroft


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28221.83022.patch
Type: text/x-patch
Size: 644 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170104/04ce65cd/attachment.bin>