<div dir="ltr">On Mon, Sep 30, 2013 at 12:34 PM, Gao, Yunzhong <span dir="ltr"><<a href="mailto:yunzhong_gao@playstation.sony.com" target="_blank">yunzhong_gao@playstation.sony.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Reid's suggestion (using wmain) does seem like a neat solution; it saves one system-locale-to-unicode conversion.<br>

I will experiment with this approach later this week to see how well it goes.<br>
I wonder whether the contents of a response file should be UTF8 or system locale encoded?<br>
<span class="HOEnZb"><font color="#888888">- Gao.<br></font></span></blockquote><div><br></div><div><br></div><div>I have implemented a solution that allows mingw, cygwin and traditional win32 to allow for unicode based paths. It will be out for review this week.  It does not use 'wmain' because that approach has serious downsides:</div>
<div>- Doesn't work at all for cygwin</div><div>- Half works with mingw</div><div><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
> -----Original Message-----<br>
> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-">llvm-commits-</a><br>
> <a href="mailto:bounces@cs.uiuc.edu">bounces@cs.uiuc.edu</a>] On Behalf Of Rafael Espíndola<br>
> Sent: Monday, September 30, 2013 12:15 PM<br>
> To: Reid Kleckner<br>
> Cc: Commit Messages and Patches for LLVM;<br>
> <a href="mailto:reviews%2BD1295%2Bpublic%2B0bd57a4d645c744a@llvm-reviews.chandlerc.com">reviews+D1295+public+0bd57a4d645c744a@llvm-reviews.chandlerc.com</a><br>
> Subject: Re: [PATCH] [3/6] Add support for non-utf8 file names in Path.inc<br>
><br>
> > Yes, probably the real solution is to define wmain instead of main, so<br>
> > we can get our hands on Unicode paths to start with, convert that to<br>
> > UTF-8 internally, and go from there.<br>
><br>
> Agreed. We really should not try guessing the encoding.<br>
><br>
> Cheers,<br>
> Rafael<br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>