On 10/25/2017 12:49 PM, Reid Kleckner wrote:
On Wed, Oct 25, 2017 at 12:14 PM, Friedman, Eli <efrie...@codeaurora.org <mailto:efrie...@codeaurora.org>> wrote:

    Maybe we should just ignore -fno-short-wchar, instead?  I think
    that's what gcc and released versions of clang do (that means
    -fno-short-wchar doesn't do anything for Windows targets, but
    that's probably fine).


No, the intention of this change was to make -fno-short-wchar do something on Windows targets.

Really?  I don't see any mention of that in either the patch itself, or in the discussion of it.  It's fine, I guess, but please add a note to the release notes.

-Eli

--
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux 
Foundation Collaborative Project

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to