RFR: 8292170: Convert CodeRootSetTable to use ResourceHashtable [v8]

Leo Korinth lkorinth at openjdk.org
Tue Jan 24 16:49:07 UTC 2023


On Tue, 24 Jan 2023 14:11:56 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Leo Korinth has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove the purge list and use the resizing of the resource hash
>>   table.
>>   
>>   Add asserts ensuring that we do not mutatet a hash table while we
>>   iterate over it. Renaming .*g1CodeCacheRemSet.?pp -> .*g1CodeRootSet.?pp
>
> src/hotspot/share/gc/g1/g1CodeRootSet.cpp line 36:
> 
>> 34: #include "runtime/atomic.hpp"
>> 35: #include "services/memTracker.hpp"
>> 36: #include "utilities/hashtable.inline.hpp"
> 
> Need to remove this include since it's no longer used.  Also, I don't see memTracker used either.  is stack.inline.hpp used ?

I will cleanup `#include`s and `class` lists

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

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


More information about the hotspot-dev mailing list