On Fri, 3 Feb 2023 04:26:40 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> Justin King has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Fix grammatical error >> >> Signed-off-by: Justin King <jck...@google.com> >> - Update based on review and remove restriction on compressed oops and >> class pointers >> >> Signed-off-by: Justin King <jck...@google.com> > > Updates look good - glad to see the flag changes go away! > > I suggest factoring out the change to > `test/jdk/jni/nullCaller/exeNullCallerTest.cpp` as it is a JDK test and not > part of hotspot. Thanks. > @dholmes-ora : Do all the sub-data tied to metaspace objects really have to > live in C-heap? @tstuefe - sorry I have no knowledge about that. Perhaps @coleenp can answer? ------------- PR: https://git.openjdk.org/jdk/pull/12229