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

[Ioi Lam]

On Tue, 12 Nov 2024 07:09:58 GMT, David Holmes <dholmes at openjdk.org> wrote:

> > Maybe we should get rid of ClassFileStream::_need_verify altogether?
> 
> Get rid of it in what way? Implicitly always true, or implicitly always false? Or?

Always true. E.g.,


  u2 get_u2(TRAPS) const {
-   if (_need_verify) {
      guarantee_more(2, CHECK_0);
-   } else {
-     assert(2 <= _buffer_end - _current, "buffer overflow");
-   }
    return get_u2_fast();
  }

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

PR Comment: https://git.openjdk.org/jdk/pull/21954#issuecomment-2469797438