On Mon, 29 Jun 2026 17:51:11 GMT, Coleen Phillimore <[email protected]> wrote:

>> This patch adds missing argument validation to  copyOfSpecialArray() and 
>> copyOfRangeSpecialArray().
>> The implementation in JVM_CopyOfSpecialArray is also updated to address a 
>> number of issues (inconsistencies, code duplication, buggy index 
>> computation). Those changes should solve JDK-8387251.
>> 
>> Tested with Mach5, tier1-4.
>> 
>> ---------
>> - [X] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> test/hotspot/jtreg/runtime/valhalla/inlinetypes/TestArrayFactories.java line 
> 412:
> 
>> 410:             throw new RuntimeException("Missing 
>> IllegalArgumentException");
>> 411:         } catch (IllegalArgumentException e) {}
>> 412:     }
> 
> Is there a positive test where a non null restricted array gets nulls for 
> indexes outside the range of from and to?  ie, the 'else' branch line 465?

Yes, in InlineTypeArray.java tests.

-------------

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2586#discussion_r3494621632

Reply via email to