[PATCH] D13363: [DeadStoreElimination] Add support for non-local DSE

[Daniel Berlin via llvm-commits]

On Wed, Nov 4, 2015 at 6:53 PM, Ivan Baev via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> ivanbaev marked 3 inline comments as done.
> ivanbaev added a comment.
>
> Hi Bruno, the patch has been rebased, and should apply cleanly on ToT.
>
> Mitch, thanks for your comments and suggestions.
> Non-local DSE extension follows the structure of HandleFree(). The
> underlying analysis for DSE is MD, and for across-BB both - HandleFree and
> HandleNonLocalDependency - use MD->getPointerDependencyFrom(). Note that MD
> is not loop-aware


What do you mean by this, precisely?


Given a store and load in the loop like this:

for (int i = 0; i < 50; ++i)
 {

load *b
store *b
}

MD will definitely give you the store as a dependency of the load.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151112/b327facb/attachment.html>