JDK 9 RFR of JDK-8132854: Adjust tier 1 and 2 definitions for nio-related intrinsics

[Paul Sandoz]

+1

Paul.

On 3 Aug 2015, at 05:19, joe darcy <joe.darcy at oracle.com> wrote:

> Hello,
> 
> With the fix for
> 
>        JDK-8076112: Add @HotSpotIntrinsicCandidate annotation to indicate methods for which Java Runtime has intrinsics
> 
> the sources for the libraries clearly indicate which methods might have intrinsic support in HotSpot, that is, which methods might have their Java source implementation replaced by some other code sequence maintained in HotSpot, such as a special-purpose processor-specific instruction.
> 
> To ease testing, it would be help if all the intrinsified methods were included in the tier 1 test group. Most of the intrinics are in java.lang, java.math, and java.util, packages whose tests are already in tier 1. Two nio-related classes are marked with @HotSpotIntrinsicCandidate:
> 
>    ./java.base/share/classes/sun/nio/cs/ISO_8859_1.java # implEncodeISOArray marked
>    ./java.base/share/classes/java/nio/Buffer.java # index check
> 
> Therefore, I propose moving the tests that directly cover those areas to tier 1:
> 
> --- a/test/TEST.groups    Mon Aug 03 09:25:02 2015 +0800
> +++ b/test/TEST.groups    Sun Aug 02 20:16:00 2015 -0700
> @@ -28,11 +28,15 @@
> tier1 = \
>     :jdk_lang \
>     :jdk_util \
> +    sun/nio/cs/ISO8859x.java \
> +    java/nio/Buffer \
>     :jdk_math
> 
> tier2 = \
>     :jdk_io \
>     :jdk_nio \
> +    -sun/nio/cs/ISO8859x.java \
> +    -java/nio/Buffer \
>     :jdk_net \
>     :jdk_time \
>     :jdk_security \
> 
> Full webrev at
> 
>    http://cr.openjdk.java.net/~darcy/8132854.0/
> 
> Thanks,
> 
> -Joe