RFR: 8292680: Convert Dictionary to ConcurrentHashTable [v5]

Coleen Phillimore coleenp at openjdk.org
Thu Aug 25 18:53:53 UTC 2022


On Thu, 25 Aug 2022 18:26:01 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review comments.
>
> src/hotspot/share/runtime/deoptimization.cpp line 962:
> 
>> 960: protected:
>> 961:   static InstanceKlass* find_cache_klass(Thread* thread, Symbol* klass_name) {
>> 962:     ResourceMark rm;
> 
> Is worth adding rm(thread) in line 962?

Yes, that's a good suggestion since we have thread.

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

PR: https://git.openjdk.org/jdk/pull/9886


More information about the hotspot-dev mailing list