Hi David,<div>IIUC Eric Feiveson drives Visual Studio visualizers. I'll email him (and will also demo to STL in Oulu). </div><div><br></div><div>Best,</div><div>Mike<br><br>On Monday, June 13, 2016, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> 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">On Mon, Jun 13, 2016 at 8:55 AM, Michael Spertus <span dir="ltr"><<a href="javascript:_e(%7B%7D,'cvml','mike@spertus.com');" target="_blank">mike@spertus.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"><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></div></div></div></blockquote><div><br></div><div>Oh, I don't use MSVC at all, so it's totally up to you, I'd just be curious if the visualizers for SmallVector were different for those of std::vector. Not that the authors of the inbuilt visualizers in MSVC have a monopoly on correct/good design here.<br><br>Might be worth roping STL (Stephan) into the thread to discuss MSVC visualizers of the STL - and/or filing a bug, if we think there are better ways to visualize containers than those provided by MSVC.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><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><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="javascript:_e(%7B%7D,'cvml','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>On Sun, Jun 12, 2016 at 7:01 PM, Mike Spertus via cfe-commits <span dir="ltr"><<a href="javascript:_e(%7B%7D,'cvml','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>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="javascript:_e(%7B%7D,'cvml','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>
</div></div></blockquote></div><br></div></div>
</blockquote></div>