<div dir="ltr">But the contents of the second iteration of the worklist could be very different based on what was skipped and what other changes happened in the first iteration.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature">~Craig</div></div>
<br><div class="gmail_quote">On Tue, Mar 21, 2017 at 5:11 PM, Daniel Berlin <span dir="ltr"><<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Mar 21, 2017 at 4:52 PM, Craig Topper via Phabricator 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">craig.topper added a comment.<br>
<br>
Yeah I saw that. Should we add a debug counter around the top level worklist loop? The semantics of the "skip" part of the counter wouldn't make sense though.<br>
<div class="m_-583643192982593670HOEnZb"><div class="m_-583643192982593670h5"><br></div></div></blockquote></span><div>That depends on what semantics you want out of it :)</div><div><br></div><div>FWIW: In NewGVN, we want the semantic of the value numbering counter to to be "choose which  instructions are processed on each iteration of the worklist".</div><div><br></div><div>There are two ways to accomplish that:</div><div>1. Record which were skipped, skip them again (some cost)</div><div>or</div><div>2. use the isCounterSet/etc to reset the counter at the start of  each iteration of the worklist to the same values. (no cost if debug is off, though i see i forgot to make this constant when #ifdef NDEBUG)</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="m_-583643192982593670HOEnZb"><div class="m_-583643192982593670h5">
<br>
<a href="https://reviews.llvm.org/D31120" rel="noreferrer" target="_blank">https://reviews.llvm.org/D3112<wbr>0</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>
</blockquote></div><br></div>