Thread (2 messages) 2 messages, 2 authors, 2015-11-18

Re: [powerpc:test 6/9] arch/sh/math-emu/math.c:129:1: warning: ISO C90 forbids mixed declarations and code

From: Joseph Myers <hidden>
Date: 2015-11-18 13:18:51
Also in: linux-sh

On Wed, 18 Nov 2015, kbuild test robot wrote:
tree:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git test
head:   ebb77ceeb18554a309846f9025a432fe848ddc8e
commit: a4572d52d6d662db3956d1bc91a745e39a376979 [6/9] sh/math-emu: Move sh from math-emu-old to math-emu
config: sh-allyesconfig (attached as .config)
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout a4572d52d6d662db3956d1bc91a745e39a376979
        # save the attached .config to linux build tree
        make.cross ARCH=sh 

All warnings (new ones prefixed by >>):

   In file included from arch/sh/math-emu/math.c:23:0:
   include/math-emu/single.h:76:21: warning: "__BIG_ENDIAN" is not defined [-Wundef]
   In file included from arch/sh/math-emu/math.c:24:0:
   include/math-emu/double.h:81:22: warning: "__BIG_ENDIAN" is not defined [-Wundef]
   arch/sh/math-emu/math.c:54:0: warning: "WRITE" redefined [enabled by default]
   include/linux/fs.h:198:0: note: this is the location of the previous definition
   arch/sh/math-emu/math.c:55:0: warning: "READ" redefined [enabled by default]
   include/linux/fs.h:197:0: note: this is the location of the previous definition
   arch/sh/math-emu/math.c: In function 'fadd':
quoted
quoted
arch/sh/math-emu/math.c:129:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
arch/sh/math-emu/math.c:129:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
   arch/sh/math-emu/math.c: In function 'fsub':
   arch/sh/math-emu/math.c:136:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
   arch/sh/math-emu/math.c:136:1: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
This looks rather like it could be some sort of false positive with the 
warning tracking / pre-existing condition given that only the warnings in 
one function are reported as new.  In any case, diagnostics with macro 
expansion tracking enabled would be more helpful.  But if genuine and not 
specific to the kernel code, I'd suggest using 
-Wno-declaration-after-statement here - as externally maintained code, it 
doesn't make sense to apply such kernel-specific coding-style rules to it.  
(It seems some other architectures disable all warnings when using this 
code.)

-- 
Joseph S. Myers
joseph@codesourcery.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help