RFR: 8358326: Use oopFactory array allocation [v2]
Frederic Parain
fparain at openjdk.org
Mon Jun 9 17:19:00 UTC 2025
On Mon, 9 Jun 2025 14:16:39 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> This patch removes cases of direct calls to {type,obj}ArrayKlass->allocate() and calls oopFactory::new_*array instead. It also renames {type,obj}ArrayKlass->allocate functions to allocate_klass and allocate_instance so it's more clear which allocation it's doing and to match InstanceKlass allocate functions, and makes these functions private with friends for Deoptimization and oopFactory. For JEP 401, arrays are being extended to support new formats and attributes and this reduces the call sites.
>> Tested with tier1-7.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>
> Sort our friends.
LGTM.
Thank you for this cleanup.
-------------
Marked as reviewed by fparain (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/25590#pullrequestreview-2910737057
More information about the hotspot-dev
mailing list