[PATCH] D79298: [NFC] Traverse function using dominator tree.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 07:59:44 PDT 2020


fhahn added a comment.

In D79298#2016937 <https://reviews.llvm.org/D79298#2016937>, @zoecarver wrote:

> @lebedev.ri you're right. I just took a look at the MSSA DSE and it looks much more powerful. I'll contribute changes to that in the future (looks like there's a lot of working being done already).


As @lebedev.ri pointed out, there's an effort to provide cross-basicblock DSE using MemorySSA. I've create an umbrella bug to track the effort: https://bugs.llvm.org/show_bug.cgi?id=45792. There are a few patches pending adding functionality, but there is at least one more feature missing (without any patches yet) to reach parity with legacy DSE: https://bugs.llvm.org/show_bug.cgi?id=45795. Please feel free to reach out if you are interested in helping out!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79298





More information about the llvm-commits mailing list