[PATCH] D116895: Fix a missed opportunity to optimize consecutive stores.

[Nadav Rotem via Phabricator via llvm-commits]

nadav updated this revision to Diff 398659.
nadav added a comment.

@hoy Thank you for the review. Yes, the code in TryToAddCandidate and CandidateMatch checks that all of the stores have the same base. I added a second test where the load aliases and prevents the store merging.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116895/new/

https://reviews.llvm.org/D116895

Files:
  llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  llvm/test/CodeGen/Hexagon/store-widen-aliased-load.ll
  llvm/test/CodeGen/PowerPC/mma-acc-memops.ll
  llvm/test/CodeGen/X86/MergeConsecutiveStores.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116895.398659.patch
Type: text/x-patch
Size: 5606 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220110/720896f1/attachment.bin>