[PATCH] D154493: [X86] Support some Intel CPUs for cpu_specific/dispatch feature

[Freddy, Ye via Phabricator via cfe-commits]

FreddyYe marked an inline comment as done.
FreddyYe added inline comments.


================
Comment at: llvm/lib/TargetParser/X86TargetParser.cpp:333
   { {"pentium"}, CK_Pentium, ~0U, FeatureX87 | FeatureCMPXCHG8B, 'B', false },
   { {"pentium-mmx"}, CK_PentiumMMX, ~0U, FeaturesPentiumMMX, '\0', false },
   { {"pentium_mmx"}, CK_PentiumMMX, ~0U, FeaturesPentiumMMX, 'D', true },
----------------
RKSimon wrote:
> add this as well?
cpu names contain '-' are not supported for cpu_specific/dispatch feature since '-' is not an identifier character. The solution here is creating a new name with '_' replacing and set the OnlyForCPUDispatchSpecific as `true`, like `icelake-client`, `icelake-server` and `goldmont-plus`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154493/new/

https://reviews.llvm.org/D154493