> 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).

Coleen Phillimore has updated the pull request incrementally with one 
additional commit since the last revision:

  Fix comments.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/32519/files
  - new: https://git.openjdk.org/jdk/pull/32519/files/9856cf76..8a14cef8

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=32519&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=32519&range=01-02

  Stats: 6 lines in 2 files changed: 2 ins; 4 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/32519.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/32519/head:pull/32519

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

Reply via email to