RFR: Split out ContinuationEntry and ContinuationWrapper classes [v2]

Coleen Phillimore coleenp at openjdk.java.net
Thu Apr 14 19:36:57 UTC 2022


> Moved ContinuationEntry and ContinuationWrapper to their own files.
> There's a bit of inconsistency because 3 ContinuationEntry functions are in ContinuationHelper_cpu.inline.hpp - maybe we should have continuationEntry_cpu.inline.hpp files for just the three.
> I'd like to change the name ContinuationEntry to ContinuationAnchor or ContinuationFrameAnchor but want to study how that would look a bit more first.
> Tested with other platforms and testing with loom-tier1 as sanity.

Coleen Phillimore has updated the pull request incrementally with three additional commits since the last revision:

 - Add continuationEntry.inline.hpp
 - Fix CONT_JFR code - moved to continuationFreezeThaw.cpp
 - Move constructors back.

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

Changes:
  - all: https://git.openjdk.java.net/loom/pull/145/files
  - new: https://git.openjdk.java.net/loom/pull/145/files/48c2662b..86423721

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=loom&pr=145&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=loom&pr=145&range=00-01

  Stats: 242 lines in 23 files changed: 55 ins; 158 del; 29 mod
  Patch: https://git.openjdk.java.net/loom/pull/145.diff
  Fetch: git fetch https://git.openjdk.java.net/loom pull/145/head:pull/145

PR: https://git.openjdk.java.net/loom/pull/145


More information about the loom-dev mailing list