RFR(XS): Trivial: 8199880: runtime/appcds/condy/CondyHelloTest.java missing at requires vm.cds

[Daniel D. Daugherty]

Thumbs up.

Dan


On 3/21/18 2:26 PM, mikhailo wrote:
> I believe this qualifies as a trivial fix. Please review.
>
>     JBS: https://bugs.openjdk.java.net/browse/JDK-8199880
>     Testing:
>         1. Ran the affected test locally (Linux-x64)  with and w/o 
> UseCompressedOops
>
>     Diff:
>         --- a/test/hotspot/jtreg/runtime/appcds/condy/CondyHelloTest.java
>         +++ b/test/hotspot/jtreg/runtime/appcds/condy/CondyHelloTest.java
>         @@ -24,7 +24,7 @@
>         /*
>         * @test
>         * @summary Make sure CDS works with a minimal test case that 
> uses a CONSTANT_Dynamic constant-pool entry
>         - * @requires os.arch != "sparcv9"
>         + * @requires (os.arch != "sparcv9") & (*vm.cds*)
>         * @modules java.base/jdk.internal.misc
>         * @library /test/lib /test/hotspot/jtreg/runtime/appcds
>         * @build CondyHello
>
>
> Thank you,
> Misha
>
>