[PATCH 5/7][MSP430][TESTSUITE] Prune messages about ISO C not supporting __int20 from output of tests

Mike Stump mikestump@comcast.net
Mon Nov 26 20:51:00 GMT 2018


On Nov 14, 2018, at 7:56 AM, Jozef Lawrynowicz <jozef.l@mittosystems.com> wrote:
> 
> Patch 5 deals with ISO C errors emitted by tests when the large memory model is
> used. size_t and ptrdiff_t are __int20 with -mlarge, and if the test is
> compiled with -pedantic-errors and -std=* or -ansi, then use of these types
> causes an error of the form:
>  ISO C does not support __int20 types
> I fixed this by adding dg-prune-output directives to tests which cause this
> error.

So, it is important that standard code not produce errors.  Kinda fundamental.

I think this should be fixed in some other way.  If a type is to be used as a standard type, producing an error for that type's use is wrong.  Instead, find a way (cough, __extension__), to mark or not warn (error) for it instead.



> 
> Alternatively, I considered adding typedefs preceded by  __extension__ to fix
> these errors, but in many cases __SIZE_TYPE__ is directly used

Can you fix __SIZE_TYPE__ itself to have __extension__ in it?  If not, then I'd find another way to remove that warning for the type.


More information about the Gcc-patches mailing list