[PATCH] D73763: [DSE] Lift post-dominance restriction.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 08:01:49 PDT 2020


fhahn abandoned this revision.
fhahn marked an inline comment as done.
fhahn added a comment.

I've put up D78932 <https://reviews.llvm.org/D78932> which implements eliminating stores to objects that are visible to the caller along the lines of  @george.burgess.iv's suggestion



================
Comment at: llvm/test/Transforms/DeadStoreElimination/MSSA/memset-missing-debugloc.ll:18
 ; int a() {
 ;   long b[]{2, 2, 2, 2, 0};
 ;   if (a())
----------------
vsapsai wrote:
> It would be great to update the comment to match the new IR, i.e., make `b` a function parameter instead of a local variable.
Thanks will do when I submit the test change!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73763





More information about the llvm-commits mailing list