<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Aug 6, 2014 at 11:32 AM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div><div class="h5"><blockquote type="cite"><div>On Aug 6, 2014, at 11:11 AM, Frédéric Riss <<a href="mailto:friss@apple.com" target="_blank">friss@apple.com</a>> wrote:</div>
<br><div><div style="word-wrap:break-word"><br><div><div>On 06 Aug 2014, at 19:50, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div><blockquote type="cite"><div style="font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
<blockquote type="cite"><blockquote type="cite">Is there<br>some more general test we can do here that would catch more cases of<br>MIs that don't have register definitions? (to have a bigger impact and<br>make the intent of the code clearer - it's not about skipping debug<br>
values, it's about skipping non-defining instructions)<br></blockquote><br>That’s a question I asked myself. If this test exists, then it could go into calculateDbgValueHistory also to get bigger impact.<br></blockquote>
<br>Yeah - then it seems weird to have it out here. Just makes the reader<br>ask more questions, I think. Either it matters and it should be in<br>calculateDbgValueHistory, as you say, or it doesn't and it shouldn't<br>
be anywhere because it's just adding cognitive load for readers...<br><br>I'd just suggest dropping that (& pestering Alexey about why it was in<br>the other place too).<br></div></blockquote></div><br><div>Well, in calculateDbgValueHistory the test has to be there as the loop is meant to handle DbgValue instructions specifically.</div>
<div><br></div></div></div></blockquote><div><br></div></div></div><div>It shouldn’t matter in collectChangingRegs(), because a debug value doesn’t define any registers. It’s probably marginally faster than going through the list of operands, but otherwise safe to drop. The other instance in calculateDbgValueHistory() needs to be there for the reason mentioned above. I’ll go ahead and commit this (without the first check) now.</div>
</div></div></blockquote><div><br></div><div>Agree. Thanks for the cleanup!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">
<div><div><br></div><div>thanks,</div><div>adrian</div><br><blockquote type="cite"><div><div style="word-wrap:break-word"><div>Fred</div></div></div></blockquote></div><br></div><br>_______________________________________________<br>

llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div>
</div></div>