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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
