On Wed, 24 Mar 2021 03:07:42 GMT, Coleen Phillimore <[email protected]> wrote:
>> src/hotspot/share/oops/instanceKlass.hpp line 468:
>>
>>> 466: void set_nest_host_index(u2 i) { _nest_host_index = i; }
>>> 467: // dynamic nest member support
>>> 468: void set_nest_host(InstanceKlass* host, JavaThread* current);
>>
>> I think we should drop the thread parameter for this method and
>> nest_host_error and avoid the tramp data. We only need the current thread
>> for a ResourceMark in a logging clause in set_nest_host, and for a
>> ConstantPoolHandle in nest_host_error. Neither of these codepaths are hot so
>> we can just manifest Thread::current() when needed there.
>> Then you can also remove the thread parameter from print_nest_host_error_on.
>
> You're right. I can remove a lot of thread arguments this way.
Done. Reran runtime/Nestmates tests and tier1.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3157
