[testsuite] Disabling gcc.dg/cpp/trad/include.c for Android

Alexander Ivchenko aivchenk@gmail.com
Tue Mar 26 12:07:00 GMT 2013


Hi,

Could you please take a look at the attached fixinclude patch
that addresses the problem:

"  We have test fail for gcc.dg/cpp/trad/include.c on Android. The
reason for that is that
-ftraditional-cpp is not expected to work on Android due to variadic
macro (like #define __builtin_warning(x, y...))
in standard headers and traditional preprocessor cannot handle them."

is it ok for trunk?

thanks,
Alexander

2013/1/9 Andrew Pinski <pinskia@gmail.com>:
> On Wed, Jan 9, 2013 at 7:14 AM, Alexander Ivchenko <aivchenk@gmail.com> wrote:
>> Hi,
>>
>>   We have test fail for gcc.dg/cpp/trad/include.c on Android. The
>> reason for that is that
>> -ftraditional-cpp is not expected to work on Android due to variadic
>> macro (like #define __builtin_warning(x, y...))
>> in standard headers and traditional preprocessor cannot handle them.
>>   The attached patch disables that test.
>
> It sounds like it is better to fix the system headers instead.  Via a
> fixincludes for older headers and have the android folks fix them for
> newer releases.
>
> Thanks,
> Andrew Pinski
-------------- next part --------------
A non-text attachment was scrubbed...
Name: trad_cpp_fixincludes_01.patch
Type: application/octet-stream
Size: 4870 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20130326/ce179353/attachment.obj>


More information about the Gcc-patches mailing list