[PATCH] D72410: [DSE] Eliminate stores by terminators (free,lifetime.end).

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 7 13:40:05 PDT 2020


fhahn updated this revision to Diff 276200.
fhahn added a comment.

Addressed comments, thanks! I updated `getLocForTerminator` to return an optional pair and removed the unnecessary `ToCheck.insert` calls.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72410

Files:
  llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
  llvm/test/Transforms/DeadStoreElimination/MSSA/2016-07-17-UseAfterFree.ll
  llvm/test/Transforms/DeadStoreElimination/MSSA/free.ll
  llvm/test/Transforms/DeadStoreElimination/MSSA/lifetime.ll
  llvm/test/Transforms/DeadStoreElimination/MSSA/memset-missing-debugloc.ll
  llvm/test/Transforms/DeadStoreElimination/MSSA/multiblock-captures.ll
  llvm/test/Transforms/DeadStoreElimination/MSSA/multiblock-malloc-free.ll
  llvm/test/Transforms/DeadStoreElimination/MSSA/simple.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72410.276200.patch
Type: text/x-patch
Size: 12860 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200707/cab9c5ce/attachment.bin>


More information about the llvm-commits mailing list