[X86] Replace PROC macros with an enum and a lookup table of processor information.
authorCraig Topper <craig.topper@intel.com>
Wed, 24 Jun 2020 17:36:02 +0000 (10:36 -0700)
committerCraig Topper <craig.topper@intel.com>
Wed, 24 Jun 2020 17:46:25 +0000 (10:46 -0700)
commit8dc92142e3c5332d741aff4bc81ee6aeed7784b2
tree574d6358e54c39b4a1c8c170432d0461d6fd3101
parentebc9e0f1f0786b892b4a6eaf50013a18aed31aa5
[X86] Replace PROC macros with an enum and a lookup table of processor information.

This patch removes the PROC macro in favor of CPUKind enum and a
table that contains information about CPUs.

The current information in the table is the CPU name, CPUKind enum
value, key feature for target multiversioning, and Is64Bit capable.
For the strings that are aliases, I've duplicated the information
in the table. This means there are more rows in the table than
CPUKind enums.

This replaces multiple StringSwitch's with loops through the table.
They are linear searches due to the table being more logically
ordered than alphabetical. The StringSwitch's would have also been
linear. I've used StringLiteral on the strings in the table so we
can quickly check the length while searching.

I contemplated having a CPUKind for each string so there was a 1:1
mapping, but didn't want to spread more names to the places that
use the enum.

My ultimate goal here is to store the features for each CPU as a
bitset within the table. Hoping to use constexpr to make this
composable so we can group features and inherit them. After the
table lookup we can turn the bitset into a list of strings for the
frontend. The current switch we have for selecting features for
CPUs has become difficult to maintain while trying to express
inheritance relationships.

Differential Revision: https://reviews.llvm.org/D82414
clang/lib/Basic/Targets/X86.cpp
llvm/include/llvm/Support/X86TargetParser.def
llvm/include/llvm/Support/X86TargetParser.h
llvm/lib/Support/X86TargetParser.cpp