This is the mail archive of the ecos-patches@sourceware.org mailing list for the eCos 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: Pedantic preprocessor


Sergei Gavrikov wrote:
> Hello list
>
> They spoke what 'untested' GCC3.4.1 (arm-elf) generates more compact
> code for float and double math than 'stable' GCC3.2.1 from ecoscentric
> ftp. I use 'stable' GCC3.2.1 and build 'new' toolchain to test that.
> That is (new code is a bit compact :). But, suddenly I found that the
> GCC3.4.1 preprocessor is more pedantic then GCC3.2.1 one. I wonder
> about CPP pedantism from other GCC suites (GCC3.4.X and GCC4.X.X). If a
> described issue the below is the GCC's 3.4.1, please, forget this. If
> other modern CPP cryes by a same way, a patch is attached.
>
> SYNOPSIS
>
> ecosconfig new sam7ex256
> ecosconfig tree
> make -s
> U CYGHWR_HAL_ARM_AT91, new inferred value AT91SAM7S
> U CYGHWR_HAL_ARM_AT91_FIQ, new inferred value 1
> U CYGHWR_HAL_ARM_AT91SAM7, new inferred value at91sam7x256
> headers finished
> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C
> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C
> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C
> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C
> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C
> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C
> build finished
>
> MEDICINE
>
> hal_arm.cdl describes 3 make rules to build arm.inc, vectors.o and
> target.ld.  CPP 'sees' all $(CFLAGS) options and cryes about the
> language specific flags. I found that it's enough to replace $(CFLASG)
> by $(ACTUAL_CFLAGS), and all backs as it was with CPP from GCC3.2.1:
>
> headers finished
> build finished
>
> I like to see these 2 lines together. And if anybody knows how to fix
> this issue in platform CDL, please, enlight me.
>   
Your patch is the correct one - I've applied it.  Next time, don't
forget the ChangeLog entry.

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------


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