Re: GDB 12.0.90 available for testing

[Eli Zaretskii <eliz@gnu.org>]

> Date: Sat, 26 Mar 2022 21:55:38 -0400
> Cc: gdb-patches@sourceware.org
> From: Simon Marchi <simon.marchi@polymtl.ca>
> 
> >        CXX    infrun.o
> >      In file included from btrace.h:30,
> > 		      from gdbthread.h:29,
> > 		      from infrun.h:21,
> > 		      from infrun.c:23:
> >      target/waitstatus.h: In function 'void stop_all_threads()':
> >      target/waitstatus.h:175:13: warning: 'ws.target_waitstatus::m_value' may be used uninitialized in this function [-Wmaybe-uninitialized]
> >        175 |     m_value = other.m_value;
> > 	   |     ~~~~~~~~^~~~~~~~~~~~~~~
> > 
> > Is this a real problem?
> 
> I think this one is not really a problem.  It complains about this code:
> 
> 	  target_waitstatus ws;
> 	  ws.set_no_resumed ();
> 	  return {NULL, minus_one_ptid, std::move (ws)};
> 
> When settnig `ws` using `set_no_resumed`, the union field ws::m_value is
> left untouched, meaning it contains random bytes.  When we move `ws`, we
> copy random bytes.  But that doesn't matter, because the destination
> target_waitstatus is of kind TARGET_WAITKIND_NO_RESUMED, so doesn't care
> about m_value.

Maybe we should initialize ws::m_value, just to shut up the compiler?
Because GDB 12 compiles cleanly other than this warning.