<div dir="ltr">+philip because he expressed concerns around limiting<div><br></div><div>+eli<div><br></div><div>Eli, i think your patch to DSE to do exact may-throw tracking is what made this worse.</div><div><br></div></div><div><a href="https://llvm.org/bugs/show_bug.cgi?id=29064">https://llvm.org/bugs/show_bug.cgi?id=29064</a><br></div><div><br></div><div>I'm not sure we should limit DSE.</div><div><br>I have an analysis pass just about ready to do the EBB/maythrow stuff, which gives "some" may-throw tracking, just not as exact as is being done here.</div><div>Rather than limit DSE, i think we should revert the more-exact tracking for now unless we can show it produces significant perf improvements.</div><div><br></div><div>Thoughts?</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 22, 2016 at 10:59 AM, David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">davidxl added a comment.<br>
<br>
The patch fixes the issue in <a href="https://llvm.org/bugs/show_bug.cgi?id=29064" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_<wbr>bug.cgi?id=29064</a><br>
<br>
<br>
<a href="https://reviews.llvm.org/D15537" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D15537</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>