RFR: 8309634: Resolve CONSTANT_MethodRef at CDS dump time [v2]
Ioi Lam
iklam at openjdk.org
Wed Jun 26 03:11:41 UTC 2024
On Mon, 24 Jun 2024 20:23:30 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:
>> Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>>
>> - Merge branch 'master' into 8309634-resolve-methods-at-dumptime
>> - @calvinccheung and @matias9927 comments
>> - Fixed whitespaces
>> - 8309634: Resolve CONSTANT_MethodRef at CDS dump time
>
> src/hotspot/share/oops/cpCache.cpp line 454:
>
>> 452:
>> 453: // Just for safety -- this should not happen, but do not archive if we ever see this.
>> 454: resolved &= !(rme->is_resolved(Bytecodes::_invokehandle) ||
>
> Don't forget to fix the whitespace problem here
Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19866#discussion_r1653862929
More information about the build-dev
mailing list