8315130: java.lang.IllegalAccessError when processing classlist to create CDS archive

[TimPushkin]

Fixes a error which significantly complicates the use of static CDS in some applications with user-provided class loaders (workarounds require either manually editing the generated CDS classlist which is error-prone, or switching to dynamic CDS which has its own implications, or rewriting the application which is not always possible).

The conflicts were trivial: a few neighboring methods missing from this version and not directly related to the change.

The PR depends on #1833 and #1834 though it can be rewritten not to if needed.

Testing besides the CI (performed on macos/arm64): runtime/cds, a minimal reproducer app described in the original bug report.

---

Progress

• Change must be properly reviewed (1 review required, with at least 1 Reviewer)
• Change must not contain extraneous whitespace
• Commit message must refer to an issue
• JDK-8315130 needs maintainer approval

Integration blocker

 ⚠️ Dependency #1834 must be integrated first

Issue

• JDK-8315130: java.lang.IllegalAccessError when processing classlist to create CDS archive (Bug - P4)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk21u-dev.git pull/1835/head:pull/1835
$ git checkout pull/1835

Update a local copy of the PR:
$ git checkout pull/1835
$ git pull https://git.openjdk.org/jdk21u-dev.git pull/1835/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 1835

View PR using the GUI difftool:
$ git pr show -t 1835

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk21u-dev/pull/1835.diff

Using Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back tpushkin! A progress list of the required criteria for merging this PR into pr/1834 will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

[openjdk]

This backport pull request has now been updated with issue from the original commit.

[mlbridge]

Webrevs

• 00: Full (281e5a5e)

[bridgekeeper]

@TimPushkin This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a /touch or /keepalive command to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

[TimPushkin]

/touch

Waiting for review

[openjdk]

@TimPushkin The pull request is being re-evaluated and the inactivity timeout has been reset.