In preparation for some Lilliput-related changes, I'd like to get some purely 
mechanical code moves out of the way. It would also improve separation of 
concerns and reduces include header bloat.

In particular, this patch does:

1) Move `CompressedKlassPointers` from `compressedOops.(cpp|hpp|inline.hpp)` to 
`compressedKlass.(cpp|hpp|inline.hpp)`

2) flatten the `NarrowPtrStruct _narrow_klass` to `address _base; int _shift` 
(its implicit null check member is not needed for Klass and it has little merit 
otherwise).

3) moved `narrowKlass` from `oopsHierarchy.hpp` to `compressedKlass.hpp`

4) remove `KlassAlignment` and `LogKlassAlignment` (the word-sized variants, 
not xxxInBytes) since they are unused

5) Move `KlassEncodingMetaspaceMax`, `LogKlassAlignmentInBytes` and 
`KlassAlignmentInBytes` to compressedKlass.hpp

6) Fixed all include issues (including existing missing includes)

7) Fixed VM struct because of (2)

Note that nothing functional is changed.

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

Commit messages:
 - JDK-8311870-Split-CompressedKlassPointers-from-compressedOops.hpp

Changes: https://git.openjdk.org/jdk/pull/14826/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14826&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8311870
  Stats: 565 lines in 35 files changed: 333 ins; 220 del; 12 mod
  Patch: https://git.openjdk.org/jdk/pull/14826.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14826/head:pull/14826

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

Reply via email to