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

[Noah Goldstein via Phabricator via llvm-commits]

goldstein.w.n added inline comments.


================
Comment at: llvm/lib/Target/X86/X86.td:1082
+  // Gracemont
+  list<SubtargetFeature> GRTTuning = [TuningMacroFusion,
+                                      TuningSlow3OpsLEA,
----------------
Imo GMT is more sensible than GRT.


================
Comment at: llvm/lib/Target/X86/X86ScheduleGRT.td:73
+// until 4 cycles after the memory operand.
+def : ReadAdvance<ReadAfterLd, 4>;
+
----------------
Is this correct? 


================
Comment at: llvm/lib/Target/X86/X86ScheduleGRT.td:537
+def : WriteRes<WriteVecMoveToGpr, [GRTPort20]> {
+  let Latency = 4;
+}
----------------
According to uops.info its 9c.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144393