RFR: 8173764: Assert in G1 BOT is wrong

[Stefan Johansson]

Thanks for the review,
Stefan

On 2017-02-23 14:46, Kim Barrett wrote:
>> On Feb 23, 2017, at 4:30 AM, Stefan Johansson <stefan.johansson at oracle.com> wrote:
>>
>> Thanks Kim,
>>
>> On 2017-02-22 21:40, Kim Barrett wrote:
>>>> On Feb 22, 2017, at 8:47 AM, Stefan Johansson <stefan.johansson at oracle.com> wrote:
>>>> […]
>>>> Updated webrevs:
>>>> Full: http://cr.openjdk.java.net/~sjohanss/8173764/hotspot.01/
>>>> Inc:  http://cr.openjdk.java.net/~sjohanss/8173764/hotspot.00-01/
>>> ------------------------------------------------------------------------------
>>> src/share/vm/gc/g1/g1BlockOffsetTable.cpp
>>>   369 #ifndef PRODUCT
>>>   370 void G1BlockOffsetTablePart::set_object_can_span(bool can_span) {
>>>   371   _object_can_span = can_span;
>>>   372 }
>>>
>>> This should be #ifdef ASSERT, and the declaration should be
>>> NOT_DEBUG_RETURN.  As written, an "optimize" build will fail.
>>>
>>> ------------------------------------------------------------------------------
>>>
>> Nice catch! I've heard that people want to remove optimized builds, after this, add me to that list :)
>>
>> I changed it as you suggested and now it builds when using optimized:
>> Full: http://cr.openjdk.java.net/~sjohanss/8173764/hotspot.02/
>> Inc: http://cr.openjdk.java.net/~sjohanss/8173764/hotspot.01-02/
>>
>> Thanks,
>> Stefan
> looks good
>