On Fri, 23 Aug 2024 06:13:29 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request with a new target base due to >> a merge or a rebase. The pull request now contains four commits: >> >> - With JDK-8338929 we don't need is_in_class_space(). >> - Merge branch 'master' into anon >> - Incorporated a set of Thomas Stuefe's comments. Take out AbstractClass >> MetaspaceObj::Type. >> - 8338526: Don't store abstract and interface Klasses in class metaspace > > src/hotspot/share/oops/klass.hpp line 205: > >> 203: >> 204: void* operator new(size_t size, ClassLoaderData* loader_data, size_t >> word_size, TRAPS) throw(); >> 205: > > Oh ArrayKlass never used this? Its good to move it to InstanceKlass. ArrayKlass has its own operator new. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/19157#discussion_r1733266271
