RFR: 8278965: crash in SymbolTable::do_lookup [v2]

[Frederic Parain]

On Fri, 10 Feb 2023 14:28:57 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Please review this small change to hold the refcount of a super class name while calling handle_parallel_super_load. Contrary to what I wrote in JDK-8298061, the refcount in the Placeholder table should never go to one.  I can reproduce this with btree012 occasionally, so this fixes this too.
>> 
>> The include in placeholders.hpp is required to make the gtest compiled since placeholders header file has Symbol::maybe_increment_refcount and decrement calls.
>> 
>> Tested with the graal test that failed, and tier1-7.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix variable name.

Looks like a reasonable fix for the issue described in the CR.

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

Marked as reviewed by fparain (Committer).

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