<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 25, 2015 at 9:19 AM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">rnk accepted this revision.<br>
rnk added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
lgtm with the updated xfail. Everything else passes.<br>
<br>
<br>
================<br>
Comment at: test/asan/TestCases/suppressions-interceptor.cc:8-9<br>
@@ -7,4 +7,4 @@<br>
<br>
 // FIXME: Windows symbolizer needs work to make this pass.<br>
 // XFAIL: android,win32<br>
<br>
----------------<br>
You can remove the win32 XFAIL and FIXME here, the test passes with this change. I must have misdiagnosed this issue when I was porting the test suite. =/<br>
<br>
The good news is that it passes. :)<br></blockquote><div><br></div><div>I have removed XFAIL and the test failed on the buildbot: <a href="http://lab.llvm.org:8011/builders/sanitizer-windows/builds/8908/steps/run%20tests/logs/stdio">http://lab.llvm.org:8011/builders/sanitizer-windows/builds/8908/steps/run%20tests/logs/stdio</a></div><div>Any ideas?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
<br>
<a href="http://reviews.llvm.org/D12305" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12305</a><br>
<br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</div></div>