RFR: 8204167: Filter out tests requiring compressed oops when CompressedOops is disabled

[coleen.phillimore at oracle.com]

Looks good!
Coleen

On 5/31/18 5:05 PM, Kim Barrett wrote:
>> On May 31, 2018, at 3:59 PM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
>>
>> On 2018-05-31 21:20, Kim Barrett wrote:
>>>> On May 31, 2018, at 8:09 AM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
>>>>
>>>> Hi all,
>>>>
>>>> Please review this patch to add @requires vm.opt.final.UseCompressedOops to those jtreg tests that requires compressed oops.
>>>>
>>>> http://cr.openjdk.java.net/~stefank/8204167/webrev.01/
>>>> https://bugs.openjdk.java.net/browse/JDK-8204167
>>>>
>>>> With this patch we now filter out tests that require compressed oops, when -XX:-UseCompressedOops is passed or ZGC is tested.
>>>>
>>>> Thanks,
>>>> StefanK
>>> test/hotspot/jtreg/runtime/CompressedOops/CompressedClassPointers.java
>>> test/hotspot/jtreg/runtime/CompressedOops/CompressedClassSpaceSize.java
>>> test/hotspot/jtreg/runtime/Metaspace/MaxMetaspaceSizeTest.java
>>>
>>> These seem to be testing UseCompressedClassPointers rather than
>>> UseCompressedOops.  Shouldn't they require UseCompressedClassPointers
>>> instead?  I know UCCP requires UCO, but requiring UCO here seems odd.
>> Yes, that makes sense. New webrevs:
>>
>> http://cr.openjdk.java.net/~stefank/8204167/webrev.02.delta/
>> http://cr.openjdk.java.net/~stefank/8204167/webrev.02/
>>
>> Thanks,
>> StefanK
> Looks good.
>