On Thu, 17 Sep 2026 13:56:11 GMT, Dan Heidinga <[email protected]> wrote:

>> [Picking up and finishing off https://github.com/openjdk/jdk/pull/32519 for 
>> Coleen]
>> 
>> Remove the upcall to addClass during class loading. The comment says it's 
>> only so GC can keep classes alive while the class loader is alive. We have 
>> other ways to do that. There were some JVMTI tests in the past that failed 
>> without this vector but today seems to be only one test. Maybe there's some 
>> code that has a dependency on this in heap walking.
>> Tested tier1-6
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Dan Heidinga has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Renamed test and removed field not present assert

Still good.

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

Marked as reviewed by sspitsyn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/32890#pullrequestreview-5239904199

Reply via email to