Integrated: 8297729: Replace GrowableArray in ComputeMoveOrder with hash table

Jorn Vernee jvernee at openjdk.org
Mon Dec 5 20:50:26 UTC 2022


On Mon, 28 Nov 2022 16:51:31 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

> Replaces the GrowableArray 'table' in ComputeMoveOrder with a real hash table.
> 
> Through testing, I found that sometimes this array is blown up to several thousand elements, most of which are `NULL`. Using a hash table prevents this large wastage.
> 
> I've touched up some of the surrounding code as well. Mostly style changes, but I've also removed the `BasicType` field from the `Move` and `MoveOperation` structs, since it was unused, and added frame data storages to the fast path for stack args as well, since both are allocated on the stack.
> 
> Testing: `jdk_foreign` test suite.

This pull request has now been integrated.

Changeset: da0917a2
Author:    Jorn Vernee <jvernee at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/da0917a2ae148ccb415accffbe1117a3dc5fdf2d
Stats:     64 lines in 2 files changed: 23 ins; 10 del; 31 mod

8297729: Replace GrowableArray in ComputeMoveOrder with hash table

Reviewed-by: coleenp, jsjolen

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

PR: https://git.openjdk.org/jdk/pull/11392


More information about the hotspot-dev mailing list