RFR: Split out ContinuationEntry and ContinuationWrapper classes [v4]

Coleen Phillimore coleenp at openjdk.java.net
Thu Apr 14 19:55:08 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 two additional commits since the last revision:

 - git add missing files
 - Add continuationEntry.inline.hpp

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

Changes:
  - all: https://git.openjdk.java.net/loom/pull/145/files
  - new: https://git.openjdk.java.net/loom/pull/145/files/6cdabe23..0fc0e746

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

  Stats: 618 lines in 10 files changed: 618 ins; 0 del; 0 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