Re: [PATCH] Fix maybe-uninitialized warnings found by gcc 9 -flto
- From
Jeff King <peff@peff.net>
- Date
- Sep 5, 2019, 22:58 UTC
- Message-ID
- <20190905225802.GA19526@sigill.intra.peff.net>
- In-Reply-To
- <963e5249-82ca-8961-917a-f45c20b1affe@gmx.net>
On Fri, Sep 06, 2019 at 12:53:49AM +0200, Stephan Beyer wrote:
Show 14 quoted lines
> On 9/6/19 12:48 AM, Jeff King wrote: > > On Thu, Sep 05, 2019 at 10:24:59AM +0200, Stephan Beyer wrote: > > > >> Compiler heuristics for detection of potentially uninitialized variables > >> may change between compiler versions and enabling link-time optimization > >> may find new warnings. Indeed, compiling with gcc 9.2.1 and enabled > >> link-time optimization feature resulted in a few hits that are fixed by > >> this patch in the most naïve way. This allows to compile git using the > >> DEVELOPER=1 switch (which sets -Werror) and using the -flto flag. > > > > Lots of discussion in this thread. Let's try to turn it into some > > patches. :) > > I thought the same and just sent my version of it. :D
Yeah, I see that our mails crossed. :) I like some of my versions better, but I'm OK with either (or a mix-and-match).
-Peff