RFR(XS): 8064786: Fix debug build after 8062808: Turn on the -Wreturn-type warning

[David Holmes]

On 13/11/2014 8:34 PM, Stefan Karlsson wrote:
> On 2014-11-13 11:20, Lindenmaier, Goetz wrote:
>> Hi,
>>
>> please review, test and sponsor this tiny change.  It fixes the debug
>> build in the gc repository.
>> https://bugs.openjdk.java.net/browse/JDK-8064786
>> http://cr.openjdk.java.net/~goetz/webrevs/8064786-warnRet/webrev.00/
>
> Looks good. Thanks for fixing. Another approach would be to just remove
> the ShouldNotReachHere() lines.
>
> I'll push when we get another review.

Reviewed.

But I'm concerned as to how this was not detected with the original fix. 
I assume we don't build with a compiler that complains about this code?

Thanks,
David

> thanks,
> StefanK
>
>>
>> Best regards,
>>    Goetz.
>