On Tue, 13 Sep 2022 13:29:45 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

> I moved misc_flags out to their own misc flags class so that we can put the 
> writeable accessFlags there too with atomic access.
> 
> Tested with tier1-4.

Nice refactoring and simplification.
It looks good to me.
Thanks,
Serguei

src/hotspot/share/prims/jvmtiRedefineClasses.cpp line 4408:

> 4406:   if (!the_class->has_been_redefined()) {
> 4407:     the_class->set_has_been_redefined();
> 4408:   }

Nit: Is this change really needed?

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

Marked as reviewed by sspitsyn (Reviewer).

PR: https://git.openjdk.org/jdk/pull/10249

Reply via email to