[PATCH] D72148: [DSE] Support traversing MemoryPhis.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 21 09:19:05 PST 2020
fhahn updated this revision to Diff 245890.
fhahn marked 3 inline comments as done.
fhahn added a comment.
In D72148#1885668 <https://reviews.llvm.org/D72148#1885668>, @asbirlea wrote:
>
j One general comment: this seems to do more than just lifting the MemoryPhi restriction. It's adding some cases that were were not added to the initial version, and that's great, but it would be good to add this in the patch description at least.
> (e.g. continue upward search in DSE.cpp:1871, adding removal of partially overlapping stores)
It looks like the overlapping store handling slipped in at some point while rebasing the series. I've removed the changes and will update the separate patch. Sorry about that.
I've extended the description to include the change to how candidates for elimination are explored now. It's quite tied to handling MemoryPhis I think, but I can split it off, if you prefer.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72148/new/
https://reviews.llvm.org/D72148
Files:
llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
llvm/test/Transforms/DeadStoreElimination/MSSA/multiblock-exceptions.ll
llvm/test/Transforms/DeadStoreElimination/MSSA/multiblock-loops.ll
llvm/test/Transforms/DeadStoreElimination/MSSA/multiblock-memoryphis.ll
llvm/test/Transforms/DeadStoreElimination/MSSA/multiblock-simple.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72148.245890.patch
Type: text/x-patch
Size: 19584 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200221/899fb1cb/attachment.bin>
More information about the llvm-commits
mailing list