On Fri, 6 May 2022 01:30:35 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Ok, moved it a bit around - now the bool field is after an int field which 
>> should make things slightly better, I guess.
>
> At the start of JavaThread we have:
> 
>  private:
>   bool           _on_thread_list;                // Is set when this 
> JavaThread is added to the Threads list
>   OopHandle      _threadObj;                     // The Java level thread 
> object
> 
> so adding it next to the existing bool seems good. The accessors don't have 
> to be defined at the same place.

Ok. Done.
This requires an extra check and a static cast to `JavaThread` in 
`CodeCache::find_blob()` but it should not be that bad.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8549

Reply via email to