RFR 8214821: Remove ConcurrentHashTable::get_copy
Robbin Ehn
robbin.ehn at oracle.com
Mon Jan 28 20:31:42 UTC 2019
Hi Gerard, sorry for the delay.
Looks good, thanks!
/Robbin
On 2019-01-23 19:49, Gerard Ziemski wrote:
> Hi all,
>
> Please review this simple fix that removes “ConcurrentHashTable::get_copy” API, whose only client was a single gtest. Doing so allows us to remove CONFIG::notfound()
>
> That gtest replaces the usage of “get_copy” API with a local “cht_get_copy” method, which itself is implemented using "ConcurrentHashTable::get” API
>
> I also took this opportunity to clean up a few method names in the test a bit.
>
> References:
>
> https://bugs.openjdk.java.net/browse/JDK-8214821
> http://cr.openjdk.java.net/~gziemski/8214821_rev1
>
> Passes gtest locally on Mac and passes Mach5 hs-tier1,2,3,4,5,6
>
>
> Cheers
>
More information about the hotspot-runtime-dev
mailing list