[cfe-dev] Patch: wchar_t support
Daniel Dunbar
daniel_dunbar at apple.com
Thu Jul 1 08:34:28 PDT 2010
On Jun 30, 2010, at 4:03 PM, Nico Weber wrote:
> On Fri, Jun 11, 2010 at 3:12 AM, John McCall <rjmccall at apple.com> wrote:
>> On Jun 10, 2010, at 9:46 PM, Nico Weber wrote:
>>> the attached patch lets @encode(wchar_t) not crash (with test case). The returned value matches g++, which returns "i" for both i386 and x86_64 archs.
>>
>> I committed this part, thanks!
>>
>>> The patch also makes one of lits error messages less confusing.
>>
>> I didn't commit this part; it looks reasonable to me, but Daniel should weigh in.
>
> Daniel, what do you think of the lit patch? Would've saved me some 5
> minutes back then.
Sure, looks fine. I would phrase it:
lit.fatal('No site specific configuration available! You may need to run 'make test' in your Clang build directory.')
- Daniel
>
>>
>> John.
More information about the cfe-dev
mailing list