<div dir="auto"><div>Sorry, I'm OOO now. I'll take a look tomorrow, thanks for catching that!<br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 14 Oct 2019, 19:20 Stella Stamenova 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">stella.stamenova added a comment.<br>
<br>
In D67347#1706405 <<a href="https://reviews.llvm.org/D67347#1706405" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D67347#1706405</a>>, @stella.stamenova wrote:<br>
<br>
> It looks like this changed fixed at least one of the XFAILed tests on Windows:<br>
><br>
> <a href="http://lab.llvm.org:8011/builders/lldb-x64-windows-ninja/builds/9751" rel="noreferrer noreferrer" target="_blank">http://lab.llvm.org:8011/builders/lldb-x64-windows-ninja/builds/9751</a><br>
><br>
> So now the test results would be red because of the unexpectedly passing test (if there wasn't another failure). Could you have a look at whether any of the other tests that were XFAILed for the same bug are also now passing and remove the expected failure tags as appropriate?<br>
<br>
<br>
This is still "failing" on the windows bot. Please fix the issue or revert the change.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D67347/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D67347/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D67347" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D67347</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>