RFR: 8369685: RISC-V: refactor code related to RVFeatureValue::enabled [v4]

Hamlin Li mli at openjdk.org
Thu Oct 16 16:17:30 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 with a new target base due to a merge or a rebase. The pull request now contains 19 commits:

 - merge master
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - use DEFAULT_VALUE for RVExtFeatureValue::value()
 - review
 - minor
 - minor
 - initial commit
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - ... and 9 more: https://git.openjdk.org/jdk/compare/95380e1e...6dd4b73f

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

Changes: https://git.openjdk.org/jdk/pull/27771/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=27771&range=03
  Stats: 46 lines in 2 files changed: 9 ins; 24 del; 13 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