RFR: 8310874: Runthese30m crashes with klass should be in the placeholders during verification

Coleen Phillimore coleenp at openjdk.org
Tue Aug 1 17:55:59 UTC 2023


On Mon, 31 Jul 2023 15:05:04 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Edit. yes this comment repeats what find_or_define_instance_class already says later on, so doesn't really add anything here.
>
> What would make more sense is find_or_define_instance_class was renamed define_instance_class (which actually handled the AllowParallelDefine logic without bothering the caller about it).

maybe not.  It does return a 'k' and the comment explains why.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14889#discussion_r1279445551


More information about the hotspot-runtime-dev mailing list