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