* sh -Werror breakage @ 2003-10-02 20:22 Andrew Cagney 2003-10-03 8:13 ` Corinna Vinschen 0 siblings, 1 reply; 5+ messages in thread From: Andrew Cagney @ 2003-10-02 20:22 UTC (permalink / raw) To: Corinna Vinschen; +Cc: gdb-patches Corinna, I'm seeing: /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ 46: warning: `flt_argreg' might be used uninitialized in this function /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ 51: warning: `reg_size' might be used uninitialized in this function Andrew ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sh -Werror breakage 2003-10-02 20:22 sh -Werror breakage Andrew Cagney @ 2003-10-03 8:13 ` Corinna Vinschen 2003-10-03 19:39 ` Andrew Cagney 2003-10-08 18:20 ` Elena Zannoni 0 siblings, 2 replies; 5+ messages in thread From: Corinna Vinschen @ 2003-10-03 8:13 UTC (permalink / raw) To: gdb-patches On Thu, Oct 02, 2003 at 04:22:50PM -0400, Andrew Cagney wrote: > Corinna, I'm seeing: > > /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ > 46: warning: `flt_argreg' might be used uninitialized in this function > /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ > 51: warning: `reg_size' might be used uninitialized in this function GCC got this wrong. I've applied a patch to shut it up. Corinna -- Corinna Vinschen Cygwin Developer Red Hat, Inc. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sh -Werror breakage 2003-10-03 8:13 ` Corinna Vinschen @ 2003-10-03 19:39 ` Andrew Cagney 2003-10-08 18:20 ` Elena Zannoni 1 sibling, 0 replies; 5+ messages in thread From: Andrew Cagney @ 2003-10-03 19:39 UTC (permalink / raw) To: gdb-patches > On Thu, Oct 02, 2003 at 04:22:50PM -0400, Andrew Cagney wrote: > >> Corinna, I'm seeing: >> >> /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ >> 46: warning: `flt_argreg' might be used uninitialized in this function >> /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ >> 51: warning: `reg_size' might be used uninitialized in this function > > > GCC got this wrong. I've applied a patch to shut it up. Don't forget to post it ;-) Andrew ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sh -Werror breakage 2003-10-03 8:13 ` Corinna Vinschen 2003-10-03 19:39 ` Andrew Cagney @ 2003-10-08 18:20 ` Elena Zannoni 2003-10-09 11:51 ` Corinna Vinschen 1 sibling, 1 reply; 5+ messages in thread From: Elena Zannoni @ 2003-10-08 18:20 UTC (permalink / raw) To: vinschen, gdb-patches Corinna Vinschen writes: > On Thu, Oct 02, 2003 at 04:22:50PM -0400, Andrew Cagney wrote: > > Corinna, I'm seeing: > > > > /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ > > 46: warning: `flt_argreg' might be used uninitialized in this function > > /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ > > 51: warning: `reg_size' might be used uninitialized in this function > > GCC got this wrong. I've applied a patch to shut it up. > > Corinna > Corinna, can you post the patch? thanks elena > -- > Corinna Vinschen > Cygwin Developer > Red Hat, Inc. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sh -Werror breakage 2003-10-08 18:20 ` Elena Zannoni @ 2003-10-09 11:51 ` Corinna Vinschen 0 siblings, 0 replies; 5+ messages in thread From: Corinna Vinschen @ 2003-10-09 11:51 UTC (permalink / raw) To: gdb-patches On Wed, Oct 08, 2003 at 02:31:22PM -0400, Elena Zannoni wrote: > Corinna Vinschen writes: > > On Thu, Oct 02, 2003 at 04:22:50PM -0400, Andrew Cagney wrote: > > > Corinna, I'm seeing: > > > > > > /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ > > > 46: warning: `flt_argreg' might be used uninitialized in this function > > > /home/cagney/PENDING/2003-09-28-deprecate-register-raw-size/src/gdb/sh-tdep.c:7\ > > > 51: warning: `reg_size' might be used uninitialized in this function > > > > GCC got this wrong. I've applied a patch to shut it up. > > > > Corinna > > > > Corinna, can you post the patch? Sure, http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/gdb/sh-tdep.c.diff?cvsroot=src&r1=1.144&r2=1.145 =================================================================== RCS file: /cvs/src/src/gdb/sh-tdep.c,v retrieving revision 1.144 retrieving revision 1.145 diff -u -r1.144 -r1.145 --- src/gdb/sh-tdep.c 2003/10/02 10:47:05 1.144 +++ src/gdb/sh-tdep.c 2003/10/03 08:13:37 1.145 @@ -743,12 +743,12 @@ { int stack_offset = 0; int argreg = ARG0_REGNUM; - int flt_argreg; + int flt_argreg = 0; int argnum; struct type *type; CORE_ADDR regval; char *val; - int len, reg_size; + int len, reg_size = 0; int pass_on_stack; /* first force sp to a 4-byte alignment */ Corinna -- Corinna Vinschen Cygwin Developer Red Hat, Inc. ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2003-10-09 11:51 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2003-10-02 20:22 sh -Werror breakage Andrew Cagney 2003-10-03 8:13 ` Corinna Vinschen 2003-10-03 19:39 ` Andrew Cagney 2003-10-08 18:20 ` Elena Zannoni 2003-10-09 11:51 ` Corinna Vinschen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox