[PATCH] D74357: [X86] Raise the latency for VectorImul from 4 to 5 in Skylake scheduler models

[Craig Topper via Phabricator via llvm-commits]

craig.topper created this revision.
craig.topper added reviewers: andreadb, RKSimon.
Herald added subscribers: gbedwell, hiraditya.
Herald added a project: LLVM.
craig.topper edited the summary of this revision.
Herald added subscribers: mstojanovic, courbet.

Based on uops.info these should have 5 cycle latency as they did on Haswell/Broadwell. I have no additional internal information from Intel.

This was also shown as a discrepancy in the spreadsheet that was sent with an early llvm-dev post about llvm-exegesis.


https://reviews.llvm.org/D74357

Files:
  llvm/lib/Target/X86/X86SchedSkylakeClient.td
  llvm/lib/Target/X86/X86SchedSkylakeServer.td
  llvm/test/tools/llvm-mca/X86/SkylakeClient/resources-avx1.s
  llvm/test/tools/llvm-mca/X86/SkylakeClient/resources-avx2.s
  llvm/test/tools/llvm-mca/X86/SkylakeClient/resources-mmx.s
  llvm/test/tools/llvm-mca/X86/SkylakeClient/resources-sse1.s
  llvm/test/tools/llvm-mca/X86/SkylakeClient/resources-sse2.s
  llvm/test/tools/llvm-mca/X86/SkylakeClient/resources-sse41.s
  llvm/test/tools/llvm-mca/X86/SkylakeClient/resources-ssse3.s
  llvm/test/tools/llvm-mca/X86/SkylakeServer/resources-avx1.s
  llvm/test/tools/llvm-mca/X86/SkylakeServer/resources-avx2.s
  llvm/test/tools/llvm-mca/X86/SkylakeServer/resources-mmx.s
  llvm/test/tools/llvm-mca/X86/SkylakeServer/resources-sse1.s
  llvm/test/tools/llvm-mca/X86/SkylakeServer/resources-sse2.s
  llvm/test/tools/llvm-mca/X86/SkylakeServer/resources-sse41.s
  llvm/test/tools/llvm-mca/X86/SkylakeServer/resources-ssse3.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74357.243669.patch
Type: text/x-patch
Size: 37306 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200210/39ac8a08/attachment-0001.bin>