[PATCH] D72148: [DSE] Support traversing MemoryPhis.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D72148#2286883 <https://reviews.llvm.org/D72148#2286883>, @ebrevnov wrote:

>> Hi @ebrevnov,
>>
>> I just wanted to briefly check in to see if you made any progress on this or if you think it is worth adding a bug report to keep track of the improvement?
>
> Hi Florian,
>
> I have a prototype which is able to handle my case. I can upload it to the phabricator for the reference. It's based on about month old sources because if I rebase to newer things stop working. Another missing piece is caching results of phi translation. In the current version phi translation is done from scratch for each new memory access. It shouldn't be hard to restructure the code to reuse phi translated address from "previous" memory access.

Yeah there have been some substantial changes to the way we find the candidates to remove, but I think the same idea should still apply. It would be ideal to update & rebase the patch, but if you won't have time to do so it would be great if you could share the older version regardless.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72148