<div dir="ltr"><div><div><div>Hi David,<br></div>While I understand the initial reasoning. I have found that this is like a hundred times better for working on Clang in practice and can't imagine working without it. The point is that many Clang data structures contain SmallVectors and having to do zero expansion clicks instead of multiple each time you take a step through the code is really helpful. If you want me to back it out and rereview we can, but I'd encourage you to try it out first.<br><br>To ask more about the aside, I'm sorry if I violated community norms. Let me tell you my reasoning, and you can clarify how I should handle in the future: Aaron approved me to do post-commit reviews on natvis changes, which I have done frequently. For this change, I wasn't putting it into phabricator because I thought pre-commit approval is required but more as a heads up. Should I change that to be if I don't feel comfortable submitting without phabricator, then do the full review process?<br><br></div><div>Thanks,<br></div><br></div>Mike<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 13, 2016 at 10:16 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</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">As for the original change proposed: My guiding principle would be "do whatever std::vector does". (& that's what I did when implementing GDB pretty printers for SmallVector/SmallString/ArrayRef, etc... )<br><br>An aside: We generally don't do time limited reviews like this. Either something needs review because you're not sure about it, or it doesn't. It sounds like the feedback you were looking for probably would've been fine a post-commit review feedback just as easily & perhaps might've been a better option. (while in this case it was fine - it's sort of a community habit/standards thing - we don't want to create the idea that lack of feedback is consent/approval in the review process)</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Sun, Jun 12, 2016 at 7:01 PM, Mike Spertus via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">mspertus closed this revision.<br>
mspertus added a comment.<br>
</div></div><div><div><br>
revision 272525<br>
<br>
<br>
<a href="http://reviews.llvm.org/D21256" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21256</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>
</blockquote></div><br></div>