<div dir="ltr">Thank you!</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 26, 2015 at 3:26 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Tue, Aug 25, 2015 at 11:55 AM, Alexey Samsonov via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</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"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><span>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></span><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" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-windows/builds/8908/steps/run%20tests/logs/stdio</a></div><div>Any ideas?</div></div></div></div></blockquote></span><div><br>The bot uses the gnuwin32 tools, which we recommend. For mostly historical reasons, I'm using the Cygwin Unix tools. Apparently the gnuwin32 version of echo produces DOS line ending text. The sanitizer suppression code wasn't expecting that. I fixed it in r246096.</div></div></div></div>
</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>