Integrated: 8365245: Move size reducing operations to GrowableArrayWithAllocator
Kim Barrett
kbarrett at openjdk.org
Sun Aug 17 13:00:16 UTC 2025
On Mon, 11 Aug 2025 13:06:01 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
> Please review this change to GrowableArray to move size reducing operations
> from GrowableArrayBase and GrowableArrayView to GrowableArrayWithAllocator.
> See JBS for rationale for this move.
>
> This is mostly just a simple code motion change, with the moved functions
> being updated to account for different name lookup rules now that they are in
> a class with a class template base class. For the most part this refactoring
> didn't require any client code changes. All but one use of one moved function
> was with a GrowableArrayWithAllocator-derived class, so not affected by moving
> the functions. The one exception was a test of ZArraySlice that was modified
> to no longer use GA::pop().
>
> Testing: mach5 tier1.
This pull request has now been integrated.
Changeset: bd65d483
Author: Kim Barrett <kbarrett at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/bd65d483df4742bb7ce79b613f10f70a45117f84
Stats: 134 lines in 2 files changed: 69 ins; 64 del; 1 mod
8365245: Move size reducing operations to GrowableArrayWithAllocator
Reviewed-by: jsjolen, stefank
-------------
PR: https://git.openjdk.org/jdk/pull/26726
More information about the hotspot-dev
mailing list