(9) RFR: 8165358: vmassert_status is not debug-only

David Holmes david.holmes at oracle.com
Mon Mar 20 03:44:00 UTC 2017


Thanks for the review Chris.

David

On 18/03/2017 11:42 PM, Chris Plummer wrote:
> Looks good. I tried it with the test that was asserting. I no longer see
> the assert with the product build and do with the debug build.
>
> Thanks for fixing!
>
> Chris
>
> On 3/18/17 12:10 AM, David Holmes wrote:
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8165358
>> webrev: http://cr.openjdk.java.net/~dholmes/8165358/webrev/
>>
>> This slipped under the radar and got deferred to JDK 10. But Chris
>> just rediscovered it and I've reclassified as a P2 that really should
>> be fixed.
>>
>> The fix is trivial and perfectly safe.
>>
>> I'll need to push this through the RDP2 approval process before
>> actually pushing, but Reviews would be appreciated asap please.
>>
>> Thanks,
>> David
>>
>


More information about the hotspot-runtime-dev mailing list