RFR: 8296157: Rename FileMapInfo::space_at() to region_at()
Ioi Lam
iklam at openjdk.org
Wed Nov 2 03:33:35 UTC 2022
On Tue, 1 Nov 2022 21:39:31 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:
> Most of the CDS code refers to a memory blocks inside a CDS file as "regions", represented by the type FileMapRegion. The method FileRegion* space_at() has been renamed to region_at() and all instances of its use have been replaced. Verified with tier 1-4 tests.
Looks good. There's more to change:
- in cds.h, change `_space` to `_regions`, and increment`CURRENT_CDS_ARCHIVE_VERSION` by 1.
- in src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/memory/FileMapInfo.java:
- `_space[0]` -> `_regions[0]`,
- `spaceField` -> `regions_0Field`
- `space_0` -> `regions_0`, etc
- This change is tested by test/hotspot/jtreg/serviceability/sa/ClhsdbCDSCore.java
- You will also see a bunch of C++ compilation errors to tell you what else need to be changed.
-------------
PR: https://git.openjdk.org/jdk/pull/10937
More information about the hotspot-runtime-dev
mailing list