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><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><br>We can iterate on it later though <br><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">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/D36347</a><br>
<br>
Files:<br>
  utils/clangdiag.py<br>
<br>
</blockquote></div>