[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