[PATCH] D98288: [DSE] Translate killing locations through phis.

[Florian Hahn via Phabricator via llvm-commits]

fhahn created this revision.
fhahn added reviewers: asbirlea, nikic, ebrevnov.
Herald added a subscriber: hiraditya.
fhahn requested review of this revision.
Herald added a project: LLVM.

This patch updates DSE to translate the address of the killing access
through phis when traversing MemoryPhis. Note that this still misses
cases where we could translate phis, but there are no MemoryPhis in the
block with the translatable phi.

This is an initial step to improve DSE in the presence of address phis.
It can be improved in the future to also do phi translation when
stepping over any type of access, not just MemoryPhis. There's also a
related patch D90095 <https://reviews.llvm.org/D90095>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98288

Files:
  llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
  llvm/test/Transforms/DeadStoreElimination/phi-translation.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98288.329451.patch
Type: text/x-patch
Size: 8803 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210309/ad7c3520/attachment.bin>