RFR (S) 8244386: convert runtime/Safepoint/AssertSafepointCheckConsistency tests to gtest

Lois Foltan lois.foltan at oracle.com
Tue Aug 25 13:17:12 UTC 2020


On 8/25/2020 8:52 AM, Coleen Phillimore wrote:
>
>
> On 8/25/20 8:30 AM, Lois Foltan wrote:
>> On 8/25/2020 8:11 AM, Coleen Phillimore wrote:
>>> This is a cool feature of gtest.  Tested with tier1 and with product 
>>> (outside of tier1), since that's a typical failure mode.
>>>
>>> open webrev at 
>>> http://cr.openjdk.java.net/~coleenp/2020/8244386.01/webrev
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8244386
>>>
>>> Thanks,
>>> Coleen
>>
>> Looks good.  Where are the AssertSafepointCheckConsistency2.java 
>> (false, false) and AssertSafepointCheckConsistency3.java (false, 
>> true) covered in the new test?
>
> Thanks Lois.  I decided that the test only needs to verify that we 
> check for the erroneous case.  The cases that pass are in the code.

Okay, I'm good with your change!
Lois

>
> Coleen
>
>>
>> Thanks,
>> Lois
>



More information about the hotspot-runtime-dev mailing list