On Mon, Jun 20, 2011 at 02:17:32PM -0700, Junio C Hamano wrote: > Andi Kleen <andi@xxxxxxxxxxxxxx> writes: > > > From: Andi Kleen <ak@xxxxxxxxxxxxxxx> > > > > Add a lot of dummy returns to silence "control flow reaches > > end of non void function" warnings with disabled noreturn. > > > > If NO_NORETURN is not disabled they will be all optimized away. > > I think this is probably a bad move, given that the previous patch is a This is basically the patch you suggested. Do you have some other suggestion now? FWIW I preferred my original minimal patch and I can go back to that one. > temporary workaround until gcc 4.6 is fixed. With -Wunreachable-code on, gcc mainline (and possibly 4.6.2) has it already fixed, but it's reasonable to assume 4.6.0 will be in use for a long time. There's nothing "temporary" about compiler workarounds, unless you wait 10 years or so. > these will introduce noise for build without NO_NORETURN (either when > profile feedback is not used, or when profile feedback build is in use and > it no longer requires the NO_NORETURN workaround). I fixed the noise in a followon patch. -Andi -- ak@xxxxxxxxxxxxxxx -- Speaking for myself only. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html