[PATCH] D144393: [X86] Add schedule module for Gracemont

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

Would this be a good time to add alderlake-p and alderlake-e cpu name aliases as well?



================
Comment at: llvm/lib/Target/X86/X86.td:1082
+  // Gracemont
+  list<SubtargetFeature> GRTTuning = [TuningMacroFusion,
+                                      TuningSlow3OpsLEA,
----------------
goldstein.w.n wrote:
> Imo GMT is more sensible than GRT.
+1 GMT is more commonly used for Gracemont already


================
Comment at: llvm/lib/Target/X86/X86.td:1576
+def : ProcModel<"gracemont", GRTModel,
+                ProcessorFeatures.ADLFeatures, ProcessorFeatures.GRTTuning>;
 
----------------
This might be better to put next to the other atom cores?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144393