[llvm] r307495 - [X86] Remove check for AVX512 support from skylake-avx512 detection in getHostCPUName.
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Sun Jul 9 00:26:14 PDT 2017
Author: ctopper
Date: Sun Jul 9 00:26:14 2017
New Revision: 307495
URL: http://llvm.org/viewvc/llvm-project?rev=307495&view=rev
Log:
[X86] Remove check for AVX512 support from skylake-avx512 detection in getHostCPUName.
Users of getHostCPUName should also use getHostCPUFeatures which will take care of making sure avx512 is disabled if the CPU doesn't support it. This is consistent with what we do for other CPUs.
Modified:
llvm/trunk/lib/Support/Host.cpp
Modified: llvm/trunk/lib/Support/Host.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Host.cpp?rev=307495&r1=307494&r2=307495&view=diff
==============================================================================
--- llvm/trunk/lib/Support/Host.cpp (original)
+++ llvm/trunk/lib/Support/Host.cpp Sun Jul 9 00:26:14 2017
@@ -681,12 +681,7 @@ getIntelProcessorTypeAndSubtype(unsigned
// Skylake Xeon:
case 0x55:
*Type = INTEL_COREI7;
- // Check that we really have AVX512
- if (Features & (1 << FEATURE_AVX512)) {
- *Subtype = INTEL_COREI7_SKYLAKE_AVX512; // "skylake-avx512"
- } else {
- *Subtype = INTEL_COREI7_SKYLAKE; // "skylake"
- }
+ *Subtype = INTEL_COREI7_SKYLAKE_AVX512; // "skylake-avx512"
break;
case 0x1c: // Most 45 nm Intel Atom processors
More information about the llvm-commits
mailing list