RFR: 8343056: C2: Micro-optimize Node lists grow
Vladimir Kozlov
kvn at openjdk.org
Fri Oct 25 18:50:05 UTC 2024
On Fri, 25 Oct 2024 09:44:23 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Seen this in Leyden profiles, but it is a generic issue. The profiles show `Node::grow` and `VectorSet::grow` as hot methods. The methods are small, and since they are implementing doubling-the-array scheme, we often exit early without actually growing. Outlining the actual growing part from the capacity check would allow inlining the common case.
>
> I'll put some performance data in the comments.
>
> Additional testing:
> - [x] Linux x86_64 server fastdebug, `tier1`
This change added `_data_size`:
https://github.com/openjdk/jdk/commit/40bbdf472c74c466393b49f52b59e78710f25c77
https://bugs.openjdk.org/browse/JDK-8234328
@cl4es what do you think?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/21707#issuecomment-2438582068
More information about the hotspot-dev
mailing list