[all-commits] [llvm/llvm-project] fa6d89: [Analysis] MemoryDepChecker::couldPreventStoreLoad...

[Simon Pilgrim via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: fa6d8977999096b2a3ae1357aa38ddf73abaf414
      https://github.com/llvm/llvm-project/commit/fa6d8977999096b2a3ae1357aa38ddf73abaf414
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-01-07 (Thu, 07 Jan 2021)

  Changed paths:
    M llvm/lib/Analysis/LoopAccessAnalysis.cpp

  Log Message:
  -----------
  [Analysis] MemoryDepChecker::couldPreventStoreLoadForward - remove dead store. NFCI.

As we're breaking from the loop when clamping MaxVF, clang static analyzer was warning that the VF iterator was being updated and never used.


  Commit: 037b058e41979fa5e6ffd209033dfe72abb97b53
      https://github.com/llvm/llvm-project/commit/037b058e41979fa5e6ffd209033dfe72abb97b53
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-01-07 (Thu, 07 Jan 2021)

  Changed paths:
    M llvm/lib/Target/AArch64/SVEIntrinsicOpts.cpp

  Log Message:
  -----------
  [AArch64] SVEIntrinsicOpts - use range loop and cast<> instead of dyn_cast<> for dereferenced pointer. NFCI.

Don't directly dereference a dyn_cast<> - use cast<> so we assert for the correct type.

Also, simplify the for loop to a range loop.

Fixes clang static analyzer warning.


Compare: https://github.com/llvm/llvm-project/compare/a2957f80f87f...037b058e4197