[all-commits] [llvm/llvm-project] a26288: [X86][Atom] Fix vector fadd/fcmp/fmul resource/thr...

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Thu May 20 10:57:29 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a26288e8030a44a5696fe170de3fea79800d7bcd
      https://github.com/llvm/llvm-project/commit/a26288e8030a44a5696fe170de3fea79800d7bcd
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-05-20 (Thu, 20 May 2021)

  Changed paths:
    M llvm/lib/Target/X86/X86ScheduleAtom.td
    M llvm/test/tools/llvm-mca/X86/Atom/resources-sse1.s
    M llvm/test/tools/llvm-mca/X86/Atom/resources-sse2.s
    M llvm/test/tools/llvm-mca/X86/Atom/resources-sse3.s
    M llvm/test/tools/llvm-mca/X86/Atom/resources-x87.s

  Log Message:
  -----------
  [X86][Atom] Fix vector fadd/fcmp/fmul resource/throughputs

Match whats documented in the Intel AOM - these are all fadd/fcmp use Port1 and fmul uses Port1, but in many cases BOTH ports are required - this was being incorrectly modelled as EITHER port.

Discovered while investigating the correct fptoui costs to fix the regressions in D101555.

Now that we can use in-order models in llvm-mca, the atom model is a good "worst case scenario" analysis for x86.




More information about the All-commits mailing list