[all-commits] [llvm/llvm-project] 45617e: [clang-doc] Pass Record argument by const-ref. NFCI.

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Fri Sep 24 02:09:01 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 45617e1dd8a1b9791f3bd15aee2614383e25b298
      https://github.com/llvm/llvm-project/commit/45617e1dd8a1b9791f3bd15aee2614383e25b298
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-09-24 (Fri, 24 Sep 2021)

  Changed paths:
    M clang-tools-extra/clang-doc/BitcodeReader.cpp

  Log Message:
  -----------
  [clang-doc] Pass Record argument by const-ref. NFCI.

Record is a SmallVector<uint64_t, 1024> - we really need to avoid passing this by value.

Avoid unnecessary big copies, reported by coverity.


  Commit: dade83c02a11401897bcdb8c255db69b5c366e95
      https://github.com/llvm/llvm-project/commit/dade83c02a11401897bcdb8c255db69b5c366e95
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-09-24 (Fri, 24 Sep 2021)

  Changed paths:
    M llvm/lib/Target/X86/X86ScheduleSLM.td
    M llvm/test/tools/llvm-mca/X86/SLM/resources-sse2.s
    M llvm/test/tools/llvm-mca/X86/SLM/resources-sse41.s

  Log Message:
  -----------
  [X86][SLM] Fix ADDQ/SUBQ/CMPEQQ throughput to account for running on either port.

Testing on a SLM box suggests these can run on either port, but the throughput is 4cy on either (inc MMX versions). Confirmed with Intel AoM / Agner / InstLatX64.


Compare: https://github.com/llvm/llvm-project/compare/0c2a4548455c...dade83c02a11


More information about the All-commits mailing list