<div dir="rtl"><div dir="ltr">The test was passing before <span style="font-size:12.8px">r247253</span><span style="font-size:12.8px"> </span>without LLVM_ON_WIN32 defined in lib/Support/Locale.cpp, so the llvm::sys::unicode::* functions were used, which may mean that the LLVM_ON_WIN32 functions are overly restrictive.</div><div dir="ltr">For example, Windows can output the UTF-8 character from <span style="font-size:12.8px">source-col-map.c correctly. Maybe the correct fix it to actually eliminate the </span>LLVM_ON_WIN32 versions.</div><div dir="ltr"><br></div><div dir="ltr">What I still don't understand - how the Windows bots were passing this test and green?</div><div dir="ltr"><br></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-09-11 16:31 GMT+03:00 Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span>:</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">Thank you for this! I wasn't certain of what the right way to fix it was. :-)<br>
<span><font color="#888888"><br>
~Aaron<br>
</font></span><div><div><br>
On Fri, Sep 11, 2015 at 9:29 AM, Yaron Keren via cfe-commits<br>
<<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
> Author: yrnkrn<br>
> Date: Fri Sep 11 08:29:12 2015<br>
> New Revision: 247410<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=247410&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=247410&view=rev</a><br>
> Log:<br>
> This test requires UTF-8 output to print the UT-8 characters.<br>
><br>
><br>
> Modified:<br>
>     cfe/trunk/test/Frontend/source-col-map.c<br>
><br>
> Modified: cfe/trunk/test/Frontend/source-col-map.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Frontend/source-col-map.c?rev=247410&r1=247409&r2=247410&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Frontend/source-col-map.c?rev=247410&r1=247409&r2=247410&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/test/Frontend/source-col-map.c (original)<br>
> +++ cfe/trunk/test/Frontend/source-col-map.c Fri Sep 11 08:29:12 2015<br>
> @@ -1,4 +1,5 @@<br>
>  // RUN: not %clang_cc1 -fsyntax-only -fmessage-length 75 -o /dev/null -x c < %s 2>&1 | FileCheck %s -strict-whitespace<br>
> +// REQUIRES: utf8-capable-terminal<br>
><br>
>  // Test case for the text diagnostics source column conversion crash.<br>
><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>