RFR: 8368897: RISC-V: Cleanup RV_EXT_FEATURE_FLAGS & RV_NON_EXT_FEATURE_FLAGS [v2]
Fei Yang
fyang at openjdk.org
Wed Oct 1 07:40:48 UTC 2025
On Tue, 30 Sep 2025 09:39:35 GMT, Hamlin Li <mli at openjdk.org> wrote:
> I'm also considering if we should remove the column ext_xx in RV_EXT_FEATURE_FLAGS, and the similar column in RV_NON_EXT_FEATURE_FLAGS in this pr, as it should be able to be generated from the pretty string. How do you think about it?
>
> The different opinions could be:
>
> * for RV_EXT_FEATURE_FLAGS, should we take for example `i` or `I` as pretty string?
The current `_features_string` on linux-riscv64 looks like:
rv64 rvi rvm rva rvf rvd rvc rvv zicboz zba zbb zbs zfa zfh zfhmin zvfh zicond
We append a `rv` prefix for `i`, `m`, `a`, `f`, `d`, `c` and `v`.
> * for RV_NON_EXT_FEATURE_FLAGS, should we use `mvendorid` or `VendorId` as pretty string?
> let me know how do you think about it.
Seems not necessary to have a pretty string for non-extension flags. Is it used anywhere?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27562#issuecomment-3355129759
More information about the hotspot-dev
mailing list