RFR: 8370225: RISC-V: cleanup verify_xxx in interp_masm_riscv.hpp [v2]

Hamlin Li mli at openjdk.org
Tue Oct 21 11:06:03 UTC 2025


On Tue, 21 Oct 2025 10:51:38 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> I think the `ASSERT`s around invocation of `verify_frame_setup` are already removed in this patch.
>
> Ah, yes for this one. Sorry, I missed that. What about `verify_access_flags`?

I also noticed `verify_access_flags`, but as all its usages are accompanied by some non-`NOT_DEBUG_RETURN` like `load_unsigned_short`, I think it's best to keep it as it is. How do you think?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27894#discussion_r2447707795


More information about the hotspot-compiler-dev mailing list