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