gracemont has some different tuning features from alderlake.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| clang/lib/Basic/Targets/X86.cpp | ||
|---|---|---|
| 551 | Why not handle this above (below tremont) as the next in the *mont series? | |
| clang/lib/Basic/Targets/X86.cpp | ||
|---|---|---|
| 551 | Good catch. This isn't a short story... In short words, it was to save codes in predefined-arch-macros.c. | |
| clang/lib/Basic/Targets/X86.cpp | ||
|---|---|---|
| 551 | Hmm - how about we move it after Tremont and then add a comment explaining that going forward atom/e-cores will share p-core defines? case CK_Tremont:
defineCPUMacros(Builder, "tremont");
break;
case CK_Gracemont:
// Gracemont and later atom-cores use p-core cpu macros.
[[fallthrough]];
case CK_Nehalem:
... | |
Address comment.
| clang/lib/Basic/Targets/X86.cpp | ||
|---|---|---|
| 551 | Looks good! I refined a little bit and updated. | |
Why not handle this above (below tremont) as the next in the *mont series?