<div dir="ltr">If folks are willing to actually commit to fixing this on a near-term timeline (and it sounds like the answer was "yes"), i'm okay with this patch.<div><br></div><div>If i've got that wrong, we really should explore other solutions (like caching).</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 23, 2016 at 4:47 PM, Philip Reames <span dir="ltr"><<a href="mailto:listmail@philipreames.com" target="_blank">listmail@philipreames.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">FYI, I'm going to defer to Danny on the choice to move forward with this patch or an alternate.  I feel he's in the best position to assess the overall timeline for MemorySSA.<div class="HOEnZb"><div class="h5"><br>
<br>
On 08/23/2016 01:35 PM, Xinliang David Li wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, Aug 23, 2016 at 1:20 PM, Daniel Berlin <<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
While implementing caching is possible, I think we should avoid<br>
reinventing the wheel in memorySSA, but to keep the fix as simple as<br>
possible.  The new limit can be adjusted higher as its meaning has<br>
changed.<br>
</blockquote>
<br>
I'd be more okay with your last argument if you were going to commit to<br>
rewriting this pass to MemorySSA.<br>
:)<br>
</blockquote>
Definitely committed -- no question about it :)  I don't think we have<br>
any choice actually, but patches like this do allow buy us more time<br>
to do so.<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If we have no-one committed to moving the pass, and no plans to implement<br>
caching, i guess my answer is "hope that someone will move this to MemorySSA<br>
one day is not a strategy".<br>
</blockquote>
that is right.<br>
<br>
For this patch, I considered it high priority because there are so<br>
many people got hit by the same issue again and again. The most recent<br>
clang hanging problem in building chromium is a recent example.<br>
<br>
David<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
</blockquote></blockquote>
<br>
</div></div></blockquote></div><br></div>