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