RFR: 8317240: Promptly free OopMapEntry after fail to insert the entry to OopMapCache

[Zhengyu Gu]

On Thu, 5 Oct 2023 12:50:11 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> In OopMapCache::lookup(), after fail to insert newly created OopMapEntry to the cache, the new entry can be free immediately, because it is not published, not accessible by other thread.
>> 
>> The patch also refactored code a bit to reduce duplicate.
>> 
>> Test:
>>    hotspot_gc on MacOSX (fastdebug and release)
>
> Yes, this seems reasonable and nice to have a function for the duplicated deallocation code.

Thanks, @coleenp and @fparain

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

PR Comment: https://git.openjdk.org/jdk/pull/15963#issuecomment-1748917994