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

Reply via email to