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