RFR: 8288623: Move Continuation classes out of javaClasses.hpp [v10]

Ioi Lam iklam at openjdk.org
Fri Jun 24 00:16:21 UTC 2022


> javaClasses.hpp is getting too big - it contains the C++ representation of over 50 Java classes.
> 
> The RFE moves the following classes into a new file, continuationJavaClasses.hpp. The naming follows the same pattern as the existing header share/jvmci/jvmciJavaClasses.hpp.
> 
> - jdk_internal_vm_ContinuationScope
> - jdk_internal_vm_Continuation
> - jdk_internal_vm_StackChunk

Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 11 commits:

 - Merge branch 'master' into 8288623-move-continuation-out-of-javaClasses-hpp
 - one more missing header
 - fixed GHA build failures
 - added missing javaClasses.inline.hpp after merge with javaThread.hpp split
 - Merge branch 'master' into 8288623-move-continuation-out-of-javaClasses-hpp
 - Review comments
 - moved to runtime/continuationJavaClasses.hpp
 - @stefank review comments
 - fixed include guard macro and copyright year
 - added missing includes that were hidden by previously unnedded includes
 - ... and 1 more: https://git.openjdk.org/jdk/compare/c8cc94a3...cd063427

-------------

Changes: https://git.openjdk.org/jdk/pull/9191/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9191&range=09
  Stats: 1064 lines in 25 files changed: 602 ins; 447 del; 15 mod
  Patch: https://git.openjdk.org/jdk/pull/9191.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9191/head:pull/9191

PR: https://git.openjdk.org/jdk/pull/9191


More information about the hotspot-dev mailing list