RFR: 8348556: Inlining fails earlier for MemorySegment::reinterpret [v5]
Per Minborg
pminborg at openjdk.org
Thu Feb 6 15:38:56 UTC 2025
> This PR proposes to add some `@ForceInline` annotations in the `Module` class in order to assist inlining of FFM var/method handles.
>
> There are also some changes in other classes (notably `j.l.Object`) which, if implemented, can take us four additional levels of inlining. However, there is a tradeoff with adding `@ForceInline` and just trying to get as deep as possible for a specific use case is probably not the best idea.
>
> So, we should discuss which of the proposed changes (if any), we'd like to integrate.
>
> Tested and passed tier1-3
Per Minborg has updated the pull request incrementally with one additional commit since the last revision:
Add more @ForceInline and a benchmark
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/23460/files
- new: https://git.openjdk.org/jdk/pull/23460/files/c095eb18..7a2f7f89
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=23460&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=23460&range=03-04
Stats: 340 lines in 8 files changed: 333 ins; 0 del; 7 mod
Patch: https://git.openjdk.org/jdk/pull/23460.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/23460/head:pull/23460
PR: https://git.openjdk.org/jdk/pull/23460
More information about the core-libs-dev
mailing list