Re: r247410 - This test requires UTF-8 output to print the UT-8 characters.

2015-09-11 Thread Yaron Keren via cfe-commits
The test was passing before r247253 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. For example, Windows can output the UTF-8 character from source-col-map.c correctly.

Re: r247410 - This test requires UTF-8 output to print the UT-8 characters.

2015-09-11 Thread Aaron Ballman via cfe-commits
Thank you for this! I wasn't certain of what the right way to fix it was. :-) ~Aaron On Fri, Sep 11, 2015 at 9:29 AM, Yaron Keren via cfe-commits wrote: > Author: yrnkrn > Date: Fri Sep 11 08:29:12 2015 > New Revision: 247410 > > URL: http://llvm.org/viewvc/llvm-project?rev=247410&view=rev > Log