On Thu, 5 May 2022 12:00:49 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> Jaroslav Bachorik has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Make sure the code blob result check is correct > > src/hotspot/share/runtime/thread.hpp line 649: > >> 647: // support ASGCT >> 648: private: >> 649: bool _in_asgct; > > The position of this field may be significant. See if there are gaps in the > Thread structure which this bool might be able to fill. Just to clarify - you mean finding a gap due to padding and putting the field there? The rest of the fields are usually clustered around the supported functionality so before I insert this flag somewhere in the middle of unrelated fields I want to be sure this is what we want. ------------- PR: https://git.openjdk.java.net/jdk/pull/8549