RFR: 8363996: Obsolete UseCompressedClassPointers [v7]
David Holmes
dholmes at openjdk.org
Fri Feb 13 05:09:25 UTC 2026
On Wed, 11 Feb 2026 17:39:33 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>>> The new `INCLUDE_CLASS_SPACE` changes look okay when viewed as their own commit, but I think they will muddy the PR for other reviewers.
>>
>> I don't understand?
>
>> > The new `INCLUDE_CLASS_SPACE` changes look okay when viewed as their own commit, but I think they will muddy the PR for other reviewers.
>>
>> I don't understand?
>
> @dholmes-ora David, I am unsure what you want me to do. Rename the macro? The macro is an integral part of the change and will be needed as long as we still support 32-bit.
@tstuefe all I was saying was that looking at the single commit that contained only the INCLUDE_CLASS_SPACE change, it was very easy to see how that change was being applied. But anyone looking at the whole PR will find it harder to see it as clearly. This is a big PR.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28366#issuecomment-3894902545
More information about the serviceability-dev
mailing list