This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


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.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]