Re: RFR: JDK-8311870: Split CompressedKlassPointers from compressedOops.hpp

2023-07-12 Thread Thomas Stuefe
On Wed, 12 Jul 2023 08:11:00 GMT, Roman Kennke wrote: > Looks good to me! Thank you @rkennke ! - PR Comment: https://git.openjdk.org/jdk/pull/14826#issuecomment-1632130286

Re: RFR: JDK-8311870: Split CompressedKlassPointers from compressedOops.hpp

2023-07-12 Thread Roman Kennke
On Tue, 11 Jul 2023 11:20:19 GMT, Thomas Stuefe 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: >

Re: RFR: JDK-8311870: Split CompressedKlassPointers from compressedOops.hpp

2023-07-11 Thread Thomas Stuefe
On Tue, 11 Jul 2023 23:33:52 GMT, Ioi Lam wrote: > Looks reasonable to me. Thank you Ioi. - PR Comment: https://git.openjdk.org/jdk/pull/14826#issuecomment-1631862898

Re: RFR: JDK-8311870: Split CompressedKlassPointers from compressedOops.hpp

2023-07-11 Thread Ioi Lam
On Tue, 11 Jul 2023 11:20:19 GMT, Thomas Stuefe 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: >

Re: RFR: JDK-8311870: Split CompressedKlassPointers from compressedOops.hpp

2023-07-11 Thread Thomas Stuefe
On Tue, 11 Jul 2023 11:20:19 GMT, Thomas Stuefe 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: >

RFR: JDK-8311870: Split CompressedKlassPointers from compressedOops.hpp

2023-07-11 Thread Thomas Stuefe
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|inli