[all-commits] [llvm/llvm-project] 13185f: [Transforms] eliminateDeadStores - remove unused v...

[Simon Pilgrim via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 13185f01546a35fb5366e50b5bb95ec3a227c5ea
      https://github.com/llvm/llvm-project/commit/13185f01546a35fb5366e50b5bb95ec3a227c5ea
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-10-14 (Thu, 14 Oct 2021)

  Changed paths:
    M llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp

  Log Message:
  -----------
  [Transforms] eliminateDeadStores - remove unused variable. NFC.

The initial MemoryAccess *Current assignment is never used, and all other uses are initialized/used within the worklist loop (and not across multiple iterations) - so move the variable internal to the loop.

Fixes scan-build unused assignment warning.


  Commit: fcbec7e668ec98f5cc12cc9b5e0b22f05ef79912
      https://github.com/llvm/llvm-project/commit/fcbec7e668ec98f5cc12cc9b5e0b22f05ef79912
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-10-14 (Thu, 14 Oct 2021)

  Changed paths:
    M llvm/lib/Target/X86/X86TargetTransformInfo.cpp

  Log Message:
  -----------
  [TTI][X86] Swap getInterleavedMemoryOpCostAVX2/getInterleavedMemoryOpCostAVX512 implementations. NFC.

I have some upcoming refactoring for SSE/AVX1 interleaving cost support, and the diff is a lot nicer if the (unaltered) AVX512 implementation isn't stuck between getInterleavedMemoryOpCost and getInterleavedMemoryOpCostAVX2


Compare: https://github.com/llvm/llvm-project/compare/b6c218d4fdb7...fcbec7e668ec