On 2013/09/20 08:05, Madhavan Srinivasan wrote: > On Friday 20 September 2013 01:34 AM, Yann E. MORIN wrote: > > Benjamin, All, > > > > On 2013-09-19 15:13 -0400, Benjamin Poirier spake thusly: > >> On 2013/09/19 19:27, Yann E. MORIN wrote: > >>> Benjamin, Madhavan, All, > >>> > >>> On 2013-09-19 11:22 -0400, Benjamin Poirier spake thusly: > >>>> On 2013/09/19 12:58, Madhavan Srinivasan wrote: > >>>>> In file included from scripts/kconfig/zconf.tab.c:2537:0: > >>>>> scripts/kconfig/menu.c: In function ‘get_symbol_str’: > >>>>> scripts/kconfig/menu.c:586:18: warning: ‘jump’ may be used uninitialized in > >>>>> this function [-Wmaybe-uninitialized] [...] > >>>> > >>>> I do not see this warning with gcc 4.7.2 or 4.8.1. Which version are you > >>>> using? > >>>> [...] > > > > No, I was not able to reproduce it with either gcc-4.4.7, gcc-4.6.4 or > > gcc-4.7.3 on my Ubuntu 13.04. Neither gcc-4.5 nor 4.8 are packaged, so > > I could not test them. > > > > Anyway, -Wmaybe-uninitialized is new with 4.7. So, the warning can't > > happen with gcc < 4.7. > > > > Madhavan, what gcc version are you using? > gcc i have is gcc version 4.8.0 20130526 In that case, I don't have a strong opinion whether the patch should be merged or not, but if it is merged, I think a comment should be added to the code or the changelog stating that this is to work around a deficiency observed in gcc 4.8.0. -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html