On Tue, 11 Jul 2023 11:20:19 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:
> 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. This pull request has now been integrated. Changeset: 753bd563 Author: Thomas Stuefe <stu...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/753bd563ecca6bb5ff9b5ebc0957bc1854dce78d Stats: 565 lines in 35 files changed: 333 ins; 220 del; 12 mod 8311870: Split CompressedKlassPointers from compressedOops.hpp Reviewed-by: iklam, rkennke ------------- PR: https://git.openjdk.org/jdk/pull/14826
