[PATCH] D52886: [X86] Move ReadAfterLd functionality into X86FoldableSchedWrite (PR36957)

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86SchedBroadwell.td:79
 
+// Vector loads are 5/5/6 cycles, so ReadAfterVec*Ld registers needn't be available
+// until 5/5/6 cycles after the memory operand.
----------------
The load latencies on BDW seem weird given that HSW and SKL, the CPUs immediately before and after are the same.


================
Comment at: lib/Target/X86/X86SchedSkylakeServer.td:74
 
-// Loads are 5 cycles, so ReadAfterLd registers needn't be available until 5
+// Integer loads are 5 cycles, so ReadAfterLd registers needn't be available until 5
 // cycles after the memory operand.
----------------
Why this comment only changed in this model? The same comment existing in SNB, HSW, BDW, and SKLClient.


Repository:
  rL LLVM

https://reviews.llvm.org/D52886