Re: eCos arm-eabi GNU tools - test release 4.6.2-20120125

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]




On 2012-03-01, Sergei Gavrikov <sergei.gavrikov@xxxxxxxxx> wrote:
> On Wed, 29 Feb 2012, Grant Edwards wrote:
>
>> Now I've got -Wno-write-strings (as seen above), but I still get 139
>> warnings, and 88 of them are because of dereferencing type-punned
>> pointers by bsd_tcpip files.
>> 
>> It still looks to me like the bsd_tcpip cdl needs to add
>> -fno-strict-aliasing to CFLAGS.
>> 
>> After doing that, I'm down to 51 warnings.
>> 
>> 39 are variables that are set but not used -- almost all in bsd_tcpip
>> files.
>> 
>> Most of the rest are signed/unsigned mismatches for pointer arguments.
>> Again, almost all are in bsd_tcpip code.
>
> Grant, I found what opens Pandora's box thanking your questions on ipv6
> stack :-) That is ipv6 code! As I could understand you include it and I
> used only ipv4 stack when I tested new toolchain. With the option
>
>   cdl_option CYGPKG_NET_INET6 {
>       user_value 1
>   };
>
> I got the same results on warnings as you have.

Ah!  That's good to know.  I was starting to wonder what I was doing
wrong. :)

For now, I've added -fno-strict-aliasing to CFLAGS for the bsd_tcpip
package.  That fixes the "type-punned pointer" warnings (which _might_
indicate real problems).  The rest of warnings look pretty harmless,
but I'm working on fixing them.   I've fixed a few "variable set-but-not
used" warnings, but there are a lot more to go.  Once I have things
cleaned up a bit, I'll work on submitting patches.

-- 
Grant Edwards               grant.b.edwards        Yow! I'm having a
                                  at               quadrophonic sensation
                              gmail.com            of two winos alone in a
                                                   steel mill!


-- 
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss



[Linux Embedded]     [U-Boot V2]     [Linux Kernel]     [Linux MIPS]     [Linux ARM]     [Linux for the Blind]     [Linux Resources]     [Photo]     [Yosemite]     [ISDN Cause Codes]     [ECOS Home]

Add to Google Powered by Linux