RFR: 8368897: RISC-V: Cleanup RV_EXT_FEATURE_FLAGS & RV_NON_EXT_FEATURE_FLAGS [v4]

Hamlin Li mli at openjdk.org
Thu Oct 9 08:37:36 UTC 2025


On Thu, 9 Oct 2025 08:07:46 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> @Hamlin-Li : Can we remove the `non_ext_` name prefix for non-extension features and maybe the `enabled()` check as well as we previously discussed?
>
> Sure, seems the prefix does not help reading the code.

> It doesn't seem to be necessary to have a enabled() check for non-extension features like zicboz_block_size, mvendorid, etc. So we might want to remove this check and give these features a default value (maybe -1 as suggested by @Hamlin-Li).

I can refactor this in another pr. In this one, I'll just focus on 2 simple things: reorder, and move comments close to feature declaration.
Same as the consistent interface of VM_Version.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27562#discussion_r2415984981


More information about the hotspot-dev mailing list