RFR: 8306582: Remove MetaspaceShared::exit_after_static_dump() [v4]
    Matias Saavedra Silva 
    matsaave at openjdk.org
       
    Tue Aug  1 21:08:05 UTC 2023
    
    
  
On Tue, 1 Aug 2023 17:02:46 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
>> Matias Saavedra Silva has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Ioi comments
>
> src/hotspot/share/oops/constantPool.cpp line 308:
> 
>> 306:           assert(java_lang_String::is_instance(obj), "must be");
>> 307:           if (!ArchiveHeapWriter::is_string_too_large_to_archive(obj)) {
>> 308:             scratch_rr->obj_at_put(i, nullptr);
> 
> Should the above be the following?
> `scratch_rr->obj_at_put(i, obj);`
Yes it should! Thank you for catching this.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14879#discussion_r1281150300
    
    
More information about the core-libs-dev
mailing list