[all-commits] [llvm/llvm-project] 882134: [TextAPI] Use const-ref iterator in for-range loop...

[Simon Pilgrim via All-commits]

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

  Changed paths:
    M llvm/include/llvm/TextAPI/InterfaceFile.h

  Log Message:
  -----------
  [TextAPI] Use const-ref iterator in for-range loop. NFCI.

Avoid unnecessary copies, reported by MSVC static analyzer.


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

  Changed paths:
    M llvm/lib/TableGen/Record.cpp

  Log Message:
  -----------
  [TableGen] Record::checkRecordAssertions() - Use const-ref iterator in for-range loop. NFCI.

Avoid unnecessary copies, reported by MSVC static analyzer.


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

  Changed paths:
    M llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h

  Log Message:
  -----------
  [AsmPrinter] DebugLocEntry::dump() - Use const-ref iterator in for-range loop. NFCI.

Avoid unnecessary copies, reported by MSVC static analyzer.


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

  Changed paths:
    M llvm/include/llvm/CodeGen/MachineInstr.h

  Log Message:
  -----------
  [CodeGen] MachineInstr::getUsedDebugRegs() - Use const-ref iterator in for-range loop. NFCI.

Avoid unnecessary copies, reported by MSVC static analyzer.


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

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

  Log Message:
  -----------
  [X86][Atom] Fix integer shuffles uops, latency and throughput

The MMX pack/unpck shuffles don't need an override - they have the same behaviour as other shuffles (Port0 only).
The SSE pslldq/psrldq shuffles don't need an override - they have the same behaviour as other shuffles (Port0 only).
The SSE pshufb shuffles use 4uops (+1 load).

Noticed the pslldq/psrldq issue while trying to improve reduction costs via the D103695 helper script, and fixed the others while reviewing. Confirmed with Intel AoM / Agner / InstLatX64.


Compare: https://github.com/llvm/llvm-project/compare/44db07f11f00...5ebe95e25673