RFR: 8369685: RISC-V: refactor code related to RVFeatureValue::enabled [v5]
Hamlin Li
mli at openjdk.org
Fri Oct 17 08:54:21 UTC 2025
> Hi,
> Can you help to review this patch?
> @RealFYang
>
> Original discussion is here: https://github.com/openjdk/jdk/pull/27562#discussion_r2415984981.
> Although seems we can not remove it, but we can still remove some of the redundant check like `mvendorid.enabled()`.
> And also do some simple refactoring related to enable/disable/enabled and so on.
>
> Thanks
Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
review
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/27771/files
- new: https://git.openjdk.org/jdk/pull/27771/files/6dd4b73f..8d65b7b8
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=27771&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=27771&range=03-04
Stats: 20 lines in 2 files changed: 11 ins; 5 del; 4 mod
Patch: https://git.openjdk.org/jdk/pull/27771.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/27771/head:pull/27771
PR: https://git.openjdk.org/jdk/pull/27771
More information about the hotspot-dev
mailing list