RFR: 8341094: Clean up relax_verify in ClassFileParser [v3]

[David Holmes]

On Fri, 8 Nov 2024 12:18:59 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> 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.

I like the update so that CFS defaults to needing verification unless CFP sets it otherwise.

-------------

PR Review: https://git.openjdk.org/jdk/pull/21954#pullrequestreview-2426143425