RFR: 8236939: [TESTBUG] Incorrect initialization in java/foreign/TestArrays.java

[Leonid Mesnik]

On 1/10/20 10:16 AM, Aleksey Shipilev wrote:
> On 1/10/20 7:06 PM, Leonid Mesnik wrote:
>> I am going to push it in jdk/jdk only and not in jdk14.
> I suspect there would be some work around x86_32 before RDP2 hits, so maybe this can go into
> jdk/jdk14 as well? This would minimize the jdk/jdk14 -> jdk/jdk merge chore next week.

Sure. The test bugs could be fixed during RDP2. So if you feel it is 
useful I will push it in jdk14.

Leonid

>
>> --- a/test/jdk/java/foreign/TestArrays.java     Wed Jan 01 03:08:45 2020 +0100
>> +++ b/test/jdk/java/foreign/TestArrays.java     Fri Jan 10 09:51:51 2020 -0800
>> @@ -76,8 +76,8 @@
>>        static VarHandle shortHandle = shorts.varHandle(short.class, PathElement.sequenceElement());
>>        static VarHandle intHandle = ints.varHandle(int.class, PathElement.sequenceElement());
>>        static VarHandle floatHandle = floats.varHandle(float.class, PathElement.sequenceElement());
>> -    static VarHandle longHandle = doubles.varHandle(long.class, PathElement.sequenceElement());
>> -    static VarHandle doubleHandle = longs.varHandle(double.class, PathElement.sequenceElement());
>> +    static VarHandle longHandle = longs.varHandle(long.class, PathElement.sequenceElement());
>> +    static VarHandle doubleHandle = doubles.varHandle(double.class, PathElement.sequenceElement());
>>    
>>        static void initBytes(MemoryAddress base, SequenceLayout seq, BiConsumer<MemoryAddress, Long> handleSetter) {
>>            for (long i = 0; i < seq.elementCount().getAsLong() ; i++) {
> This looks good to me. It does look trivial.
>