<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Oct 26, 2017 at 2:48 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Seems fine, it would be nice if the workflow could be improved a little bit so that all you have to do is say `clangdiag break —error=“-Wcovered-switch”` or something .  I think that gives the most intuitive usage for people, even it’s a bit harder to implement.<br></blockquote><div><br></div><div>The idea was to break on actual diagnostics emitted, but if you want to break on diagnostic usage, i.e., when it was checked but not emitted, I suppose that's possible as well.  diagtool doesn't produce a mapping for this, but it could be added -- assuming tablegen produced enough info in the .inc files to support it.  I added the feature I'm using here a few months ago, which was an extension to what Alex added earlier.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>I also think user shouldn’t really have to concern themselves with diagtool, it should all just be magic.  I get why it’s easier to do this way, but from the users perspective, having the commands map as closely as possible to the thing the person wants to do and hiding implementation details is a big win from a usability standpoint.<br></blockquote><div><br></div><div>For the normal use case, i.e., clang/llvm developers that build both together, it will just work by magic, i.e., you just run enable/disable.  The only problem is when you build out-of-tree.  If you can suggest a way to find the correct location by examining the executable, I'd be happy to add it.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>We can iterate on it later though <br></blockquote><div><br></div><div>I'm happy to keep hacking on it -- got plenty of time on my hands right now...  And I get to learn more about lldb...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><div class="gmail_quote"><div dir="ltr">On Thu, Oct 26, 2017 at 2:38 PM Don Hinton via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hintonda updated this revision to Diff 120492.<br>
hintonda added a comment.<br>
<br>
- Remove debugging print statement, and enhance help message.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D36347" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D36347</a><br>
<br>
Files:<br>
  utils/clangdiag.py<br>
<br>
</blockquote></div>
</div></div></blockquote></div><br></div></div>