Messages in this thread | | | From | Christophe Leroy <> | Subject | Re: arch/powerpc/math-emu/fsel.c:14:2: warning: variable '_fex' set but not used | Date | Sat, 8 Jul 2023 10:44:25 +0000 |
| |
Le 07/07/2023 à 23:03, kernel test robot a écrit : > Hi Christophe, > > First bad commit (maybe != root cause): > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > head: 4f6b6c2b2f86b7878a770736bf478d8a263ff0bc > commit: 7245fc5bb7a966852d5bd7779d1f5855530b461a powerpc/math-emu: Remove -w build flag and fix warnings > date: 10 months ago > config: powerpc-randconfig-r033-20230708 (https://download.01.org/0day-ci/archive/20230708/202307080446.rpYEMqQ2-lkp@intel.com/config) > compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a) > reproduce: (https://download.01.org/0day-ci/archive/20230708/202307080446.rpYEMqQ2-lkp@intel.com/reproduce)
That's a bug in CLANG, see https://github.com/llvm/llvm-project/issues/63315
Thanks Christophe
> > If you fix the issue in a separate patch/commit (i.e. not just a new version of > the same patch/commit), kindly add following tags > | Reported-by: kernel test robot <lkp@intel.com> > | Closes: https://lore.kernel.org/oe-kbuild-all/202307080446.rpYEMqQ2-lkp@intel.com/ > > All warnings (new ones prefixed by >>): > >>> arch/powerpc/math-emu/fsel.c:14:2: warning: variable '_fex' set but not used [-Wunused-but-set-variable] > 14 | FP_DECL_EX; > | ^ > include/math-emu/soft-fp.h:94:24: note: expanded from macro 'FP_DECL_EX' > 94 | #define FP_DECL_EX int _fex = 0 > | ^ > arch/powerpc/math-emu/fsel.c:11:1: warning: no previous prototype for function 'fsel' [-Wmissing-prototypes] > 11 | fsel(u32 *frD, void *frA, u32 *frB, u32 *frC) > | ^ > arch/powerpc/math-emu/fsel.c:10:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > 10 | int > | ^ > | static > 2 warnings generated. > -- >>> arch/powerpc/math-emu/fcmpu.c:13:2: warning: variable 'A_c' set but not used [-Wunused-but-set-variable] > 13 | FP_DECL_D(A); > | ^ > include/math-emu/double.h:73:23: note: expanded from macro 'FP_DECL_D' > 73 | #define FP_DECL_D(X) _FP_DECL(2,X) > | ^ > include/math-emu/op-common.h:28:14: note: expanded from macro '_FP_DECL' > 28 | _FP_I_TYPE X##_c=0, X##_s=0, X##_e=0; \ > | ^ > <scratch space>:171:1: note: expanded from here > 171 | A_c > | ^ >>> arch/powerpc/math-emu/fcmpu.c:14:2: warning: variable 'B_c' set but not used [-Wunused-but-set-variable] > 14 | FP_DECL_D(B); > | ^ > include/math-emu/double.h:73:23: note: expanded from macro 'FP_DECL_D' > 73 | #define FP_DECL_D(X) _FP_DECL(2,X) > | ^ > include/math-emu/op-common.h:28:14: note: expanded from macro '_FP_DECL' > 28 | _FP_I_TYPE X##_c=0, X##_s=0, X##_e=0; \ > | ^ > <scratch space>:177:1: note: expanded from here > 177 | B_c > | ^ >>> arch/powerpc/math-emu/fcmpu.c:15:2: warning: variable '_fex' set but not used [-Wunused-but-set-variable] > 15 | FP_DECL_EX; > | ^ > include/math-emu/soft-fp.h:94:24: note: expanded from macro 'FP_DECL_EX' > 94 | #define FP_DECL_EX int _fex = 0 > | ^ > arch/powerpc/math-emu/fcmpu.c:11:1: warning: no previous prototype for function 'fcmpu' [-Wmissing-prototypes] > 11 | fcmpu(u32 *ccr, int crfD, void *frA, void *frB) > | ^ > arch/powerpc/math-emu/fcmpu.c:10:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > 10 | int > | ^ > | static > 4 warnings generated. > -- >>> arch/powerpc/math-emu/fctiwz.c:14:2: warning: variable '_fex' set but not used [-Wunused-but-set-variable] > 14 | FP_DECL_EX; > | ^ > include/math-emu/soft-fp.h:94:24: note: expanded from macro 'FP_DECL_EX' > 94 | #define FP_DECL_EX int _fex = 0 > | ^ > arch/powerpc/math-emu/fctiwz.c:11:1: warning: no previous prototype for function 'fctiwz' [-Wmissing-prototypes] > 11 | fctiwz(u32 *frD, void *frB) > | ^ > arch/powerpc/math-emu/fctiwz.c:10:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > 10 | int > | ^ > | static > 2 warnings generated. > -- >>> arch/powerpc/math-emu/fctiw.c:14:2: warning: variable '_fex' set but not used [-Wunused-but-set-variable] > 14 | FP_DECL_EX; > | ^ > include/math-emu/soft-fp.h:94:24: note: expanded from macro 'FP_DECL_EX' > 94 | #define FP_DECL_EX int _fex = 0 > | ^ > arch/powerpc/math-emu/fctiw.c:11:1: warning: no previous prototype for function 'fctiw' [-Wmissing-prototypes] > 11 | fctiw(u32 *frD, void *frB) > | ^ > arch/powerpc/math-emu/fctiw.c:10:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > 10 | int > | ^ > | static > 2 warnings generated. > > > vim +/_fex +14 arch/powerpc/math-emu/fsel.c > > ^1da177e4c3f41 arch/ppc/math-emu/fsel.c Linus Torvalds 2005-04-16 9 > ^1da177e4c3f41 arch/ppc/math-emu/fsel.c Linus Torvalds 2005-04-16 10 int > ^1da177e4c3f41 arch/ppc/math-emu/fsel.c Linus Torvalds 2005-04-16 11 fsel(u32 *frD, void *frA, u32 *frB, u32 *frC) > ^1da177e4c3f41 arch/ppc/math-emu/fsel.c Linus Torvalds 2005-04-16 12 { > ^1da177e4c3f41 arch/ppc/math-emu/fsel.c Linus Torvalds 2005-04-16 13 FP_DECL_D(A); > d2b194ed820880 arch/powerpc/math-emu/fsel.c Kumar Gala 2008-06-04 @14 FP_DECL_EX; > ^1da177e4c3f41 arch/ppc/math-emu/fsel.c Linus Torvalds 2005-04-16 15 > > :::::: The code at line 14 was first introduced by commit > :::::: d2b194ed820880eb19c43b9c10d9f5f30026ee54 powerpc/math-emu: Use kernel generic math-emu code > > :::::: TO: Kumar Gala <galak@kernel.crashing.org> > :::::: CC: Kumar Gala <galak@kernel.crashing.org> >
| |