RFR: 8341094: Clean up relax_verify in ClassFileParser [v2]
Coleen Phillimore
coleenp at openjdk.org
Thu Nov 7 23:03:26 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:
Rename ClassFileStream::_need_verify to _check_truncation and set based on one call in ClassFileParser.
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/21954/files
- new: https://git.openjdk.org/jdk/pull/21954/files/6f74df44..c8597cbf
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=21954&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=21954&range=00-01
Stats: 30 lines in 8 files changed: 0 ins; 7 del; 23 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