RFR: 8306460: Clear JVM_ACC_QUEUED flag on methods when dumping dynamic CDS archive [v2]

Ashutosh Mehra duke at openjdk.org
Mon May 8 13:16:30 UTC 2023


On Tue, 25 Apr 2023 22:10:04 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Ashutosh Mehra 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 three additional commits since the last revision:
>> 
>>  - Remove unshareable flags in Method and InstanceKlass
>>    
>>    Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>
>>  - Merge branch 'master' of github.com:openjdk/jdk into JDK-8306460
>>  - 8306460: Clear JVM_ACC_QUEUED flag on methods when dumping dynamic CDS archive
>>    
>>    Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>
>
> Yes, you're right, all these flags shouldn't be in the archive.  I have a patch for JDK-8306851 which will make it easier to unset all of these flags (except has_loops/has_loops_init, which we want set in the archive).  Maybe this change should wait.

@coleenp I have updated this PR with additional commit to clear other flags as well and, as mentioned in my previous comment, added asserts for `is_old`, `is_obsolete`, and `is_deleted`. Can you please review.

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

PR Comment: https://git.openjdk.org/jdk/pull/13652#issuecomment-1538342372


More information about the hotspot-dev mailing list