RFR: 8355013: GrowableArray default constructor should not allocate [v2]

Quan Anh Mai qamai at openjdk.org
Fri Apr 18 08:47:41 UTC 2025


On Fri, 18 Apr 2025 07:08:45 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   refine tests
>
> test/hotspot/gtest/utilities/test_growableArray.cpp line 446:
> 
>> 444: // static empty vector
>> 445: const GrowableArray<int> empty_array(mtTest);
>> 446: const GrowableArrayCHeap<int, mtTest> empty_cheap_array;
> 
> Where are these used?

Thanks for your reviews, these are additional to ensure that `GrowableArray` instances can be created in static context. I have added a test to check these are truly empty.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24748#discussion_r2050364560


More information about the hotspot-dev mailing list