RFR: 8338526: Don't store abstract and interface Klasses in class metaspace [v4]
Coleen Phillimore
coleenp at openjdk.org
Thu Aug 29 11:40:27 UTC 2024
On Thu, 29 Aug 2024 05:24:18 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Merge branch 'anon' of github.com:coleenp/jdk into anon
>> - Fix copyright
>
> src/hotspot/share/jfr/recorder/checkpoint/types/traceid/jfrTraceIdKlassQueue.cpp line 79:
>
>> 77:
>> 78: static bool can_compress_element(const Klass* klass) {
>> 79: return Metaspace::is_in_class_space(klass) &&
>
> Suggestion:
>
> return (Metaspace::is_in_class_space(klass) || Metaspace::is_in_shared_metaspace(klass)) &&
Is this right? If UseCompressedClassPointers is off, then the shared metaspace isn't in compressed space?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19157#discussion_r1736041738
More information about the core-libs-dev
mailing list