<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 4, 2015 at 6:53 PM, Ivan Baev via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">ivanbaev marked 3 inline comments as done.<br>
ivanbaev added a comment.<br>
<br>
Hi Bruno, the patch has been rebased, and should apply cleanly on ToT.<br>
<br>
Mitch, thanks for your comments and suggestions.<br>
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 </blockquote><div><br></div><div>What do you mean by this, precisely?</div><div><br></div><div><br></div><div>Given a store and load in the loop like this:<br><br></div><div>for (int i = 0; i < 50; ++i)</div><div> {</div><div><br></div><div>load *b<br></div><div>store *b</div><div>}</div><div><br></div><div>MD will definitely give you the store as a dependency of the load.</div><div><br></div></div></div></div>