On Mon, 20 Nov 2023 22:08:49 GMT, Coleen Phillimore <[email protected]> wrote:
>> Jaroslav Bachorik has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add exhaustive check for method holder availability (1)
>
> src/hotspot/share/classfile/classFileParser.cpp line 5579:
>
>> 5577:
>> 5578: if (_methods != nullptr) {
>> 5579: // Free methods - those methods are not fully wired and miss the
>> method holder
>
> How about saying: for methods whose InstanceKlass as method holder is not yet
> created?
And is now back, but I applied your suggestion, @coleenp
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16662#discussion_r1403513054
