RFR: 8329433: Reduce nmethod header size [v5]
Vladimir Kozlov
kvn at openjdk.org
Tue Apr 16 18:36:11 UTC 2024
> This is part of changes which try to reduce size of `nmethod` and `codeblob` data vs code in CodeCache.
> These changes reduced size of `nmethod` header from 288 to 232 bytes. From 304 to 248 in optimized VM:
>
> Statistics for 1282 bytecoded nmethods for C2:
> total in heap = 5560352 (100%)
> header = 389728 (7.009053%)
>
> vs
>
> Statistics for 1322 bytecoded nmethods for C2:
> total in heap = 8307120 (100%)
> header = 327856 (3.946687%)
>
>
> Several unneeded fields in `nmethod` and `CodeBlob` were removed. Some fields were changed from `int` to `int16_t` with added corresponding asserts to make sure their values are fit into 16 bits.
>
> I did additional cleanup after recent `CompiledMethod` removal.
>
> Tested tier1-7,stress,xcomp and performance testing.
Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
Address comments. Used checked_cast.
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/18768/files
- new: https://git.openjdk.org/jdk/pull/18768/files/6cb22e81..8405a23d
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=18768&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=18768&range=03-04
Stats: 32 lines in 3 files changed: 4 ins; 11 del; 17 mod
Patch: https://git.openjdk.org/jdk/pull/18768.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/18768/head:pull/18768
PR: https://git.openjdk.org/jdk/pull/18768
More information about the serviceability-dev
mailing list