Re: [PATCH] Avoid uninitialized warning in i386.c

2013-12-18 Thread Jakub Jelinek
On Wed, Dec 18, 2013 at 04:27:23PM +0100, Marek Polacek wrote: > Okay. Regtested/bootstrapped on x86_64-linux, ok now? > > 2013-12-18 Marek Polacek > > * config/i386/i386.c (ix86_parse_stringop_strategy_string): Remove > variable alg. Use index variable i directly. Yes, thanks.

Re: [PATCH] Avoid uninitialized warning in i386.c

2013-12-18 Thread Marek Polacek
On Wed, Dec 18, 2013 at 02:45:12PM +0100, Jakub Jelinek wrote: > On Wed, Dec 18, 2013 at 02:39:44PM +0100, Marek Polacek wrote: > > Bootstrap with -fsanitize=undefined revealed that the alg variable > > may be used uninitialized. Or at least gcc thinks so. This patch > > initializes it to 0. > >

Re: [PATCH] Avoid uninitialized warning in i386.c

2013-12-18 Thread Jakub Jelinek
On Wed, Dec 18, 2013 at 02:39:44PM +0100, Marek Polacek wrote: > Bootstrap with -fsanitize=undefined revealed that the alg variable > may be used uninitialized. Or at least gcc thinks so. This patch > initializes it to 0. > > Regtested/bootstrapped on x86_64-linux, ok for trunk? Do stri