That's possible that we haven't defined the macro correctly. But at the same
time it's not a good idea for the test to depend on the system headers to
work. For the purpose of this test, this may be necessary. I would still
like to disable the test for Hexagon since we're not planning to support
C++11.

Thanks,
Jyotsna
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation


>-----Original Message-----
>From: [email protected] [mailto:[email protected]] On Behalf Of
>Richard Smith
>Sent: Tuesday, July 02, 2013 12:17 PM
>To: Jyotsna Verma
>Cc: [email protected]
>Subject: Re: XFAIL Headers/cxx11.cpp for Hexagon
>
>On Tue, Jul 2, 2013 at 10:14 AM, Richard Smith <[email protected]>
>wrote:
>> On Tue, Jul 2, 2013 at 9:14 AM, Jyotsna Verma <[email protected]>
>wrote:
>>> The attached patch XFAILs Headers/cxx11.cpp test for Hexagon as it
>>> doesn't support C++11 and the test depends on some system headers.
>>
>> What's the exact cause of the failure? This sounds like the root cause
>> is probably that Hexagon is incorrectly defining __STDC_HOSTED__.
>
>Hmm, if it's only the -fmodules part of the test that's failing, then
perhaps
>that's a different issue (and we should either disable only that half of
the test,
>or fix whichever of our headers doesn't work for Hexagon).

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to