RFR: 8297729: Replace GrowableArray in ComputeMoveOrder with hash table [v4]
Jorn Vernee
jvernee at openjdk.org
Mon Dec 5 17:48:18 UTC 2022
> 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.
Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
Use specialized equals function
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/11392/files
- new: https://git.openjdk.org/jdk/pull/11392/files/40e66a7c..22ae2770
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=11392&range=03
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=11392&range=02-03
Stats: 10 lines in 1 file changed: 6 ins; 3 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/11392.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/11392/head:pull/11392
PR: https://git.openjdk.org/jdk/pull/11392
More information about the hotspot-dev
mailing list