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:03:39 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> This comment is still true.  non bootstrap parallel-capable class loaders might return a different k.
>
> 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).

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

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


More information about the hotspot-runtime-dev mailing list