<div dir="ltr">LGTM<div><br></div><div> - Daniel</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 31, 2014 at 11:57 PM, Eric Fiselier <span dir="ltr"><<a href="mailto:eric@efcs.ca" target="_blank">eric@efcs.ca</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi ddunbar,<br>
<br>
This patch add a --show-xfail flag. If this flag is specified then each xfail test will be printed to output.<br>
When it is not given xfail tests are ignored. Ignoring xfail tests is the current behavior.<br>
<br>
This flag is meant to mirror the --show-unsupported flag that was recently added.<br>
<br>
<a href="http://reviews.llvm.org/D4750" target="_blank">http://reviews.llvm.org/D4750</a><br>
<br>
Files:<br>
  docs/CommandGuide/lit.rst<br>
  utils/lit/lit/main.py<br>
<br>
Index: docs/CommandGuide/lit.rst<br>
===================================================================<br>
--- docs/CommandGuide/lit.rst<br>
+++ docs/CommandGuide/lit.rst<br>
@@ -88,6 +88,10 @@<br>
<br>
  Show the names of unsupported tests.<br>
<br>
+.. option:: --show-xfail<br>
+<br>
+ Show the names of tests that were expected to fail.<br>
+<br>
 .. _execution-options:<br>
<br>
 EXECUTION OPTIONS<br>
Index: utils/lit/lit/main.py<br>
===================================================================<br>
--- utils/lit/lit/main.py<br>
+++ utils/lit/lit/main.py<br>
@@ -44,6 +44,7 @@<br>
<br>
         shouldShow = test.result.code.isFailure or \<br>
             (self.opts.show_unsupported and <a href="http://test.result.code.name" target="_blank">test.result.code.name</a> == 'UNSUPPORTED') or \<br>
+            (self.opts.show_xfail and <a href="http://test.result.code.name" target="_blank">test.result.code.name</a> == 'XFAIL') or \<br>
             (not self.opts.quiet and not self.opts.succinct)<br>
         if not shouldShow:<br>
             return<br>
@@ -173,6 +174,9 @@<br>
     group.add_option("", "--show-unsupported", dest="show_unsupported",<br>
                      help="Show unsupported tests",<br>
                      action="store_true", default=False)<br>
+    group.add_option("", "--show-xfail", dest="show_xfail",<br>
+                     help="Show tests that were expected to fail",<br>
+                     action="store_true", default=False)<br>
     parser.add_option_group(group)<br>
<br>
     group = OptionGroup(parser, "Test Execution")<br>
</blockquote></div><br></div>