RFR: 8341094: Clean up relax_verify in ClassFileParser [v3]
Coleen Phillimore
coleenp at openjdk.org
Fri Nov 8 12:18:59 UTC 2024
> Removed ClassFileParser::_relax_verify and cleaned up setting need_verify in the ClassFileStream code and ClassFileParser. Wrote up how I found control to flow to this setting with -Xverify:all/none/remote settings, which become BytecodeVerificationRemote/Local in the CR comments.
>
> Tested with tier1-4. Valid class, field and method name testing is done through the JCK and there are CDS tests that test verification in runtime/cds/appcds/VerifierTest.java.
Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
Unexpose the ClassFileStream's need_verify field. It's used internally based on what ClassFileParser gets for the answer based on the BytecodeVerificationnRemote/Local flags and the class loader.
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/21954/files
- new: https://git.openjdk.org/jdk/pull/21954/files/c8597cbf..8c1c3e3e
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=21954&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=21954&range=01-02
Stats: 34 lines in 11 files changed: 0 ins; 12 del; 22 mod
Patch: https://git.openjdk.org/jdk/pull/21954.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/21954/head:pull/21954
PR: https://git.openjdk.org/jdk/pull/21954
More information about the hotspot-runtime-dev
mailing list