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