__atomic_futex_unsigned::_M_load_when_not_equal has a 'return' statement with no value, in function returning unsigned int

[Torvald Riegel]

On Thu, 2015-07-23 at 09:21 +0100, Jonathan Wakely wrote:
> On 22/07/15 22:30 -0400, Patrick Palka wrote:
> >Specifically, line 149 of atomix_futex.h has a bare "return;"
> >statement, but the function is marked as returning non-void.  This was
> >caught while working on PR c++/18969.  Vanilla G++ does not catch this
> >error because return statements inside templates are currently only
> >analyzed during instantiation time.
> 
> Ouch. Luckily that function is never called, so it's not doing any
> harm.
> 
> Torvald, assuming we want to keep that unused function, is this the
> right fix?

I think so.  Thanks!