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

Hamlin Li mli at openjdk.org
Wed Oct 1 10:26:54 UTC 2025


> Hi,
> Can you help to review the patch?
> 
> This patch cleans up RV_EXT_FEATURE_FLAGS & RV_NON_EXT_FEATURE_FLAGS, as discussed https://github.com/openjdk/jdk/pull/27152#discussion_r2367109820:
> * reorder flags in alphabetic order for RV_EXT_FEATURE_FLAGS
> * move comments close to feature declaration for RV_EXT_FEATURE_FLAGS & RV_NON_EXT_FEATURE_FLAGS
> 
> We also discussed (https://github.com/openjdk/jdk/pull/27171#discussion_r2387195562) the assert introduced in https://github.com/openjdk/jdk/pull/24094, previously we think this will restrict the flags order in RV_EXT_FEATURE_FLAGS, but I found out that this  assert (<del>is not necessary</del>, so we should be able to order flags in RV_EXT_FEATURE_FLAGS in any way we'd like to) does not work as expected, will fix this in another pr.
> 
> 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 14 commits:

 - merge master
 - Merge branch 'openjdk:master' into master
 - remove NAME
 - merge master
 - Merge branch 'openjdk:master' into master
 - initial commit
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - ... and 4 more: https://git.openjdk.org/jdk/compare/5a2700f2...f0701a7d

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

Changes: https://git.openjdk.org/jdk/pull/27562/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=27562&range=02
  Stats: 193 lines in 6 files changed: 42 ins; 46 del; 105 mod
  Patch: https://git.openjdk.org/jdk/pull/27562.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/27562/head:pull/27562

PR: https://git.openjdk.org/jdk/pull/27562


More information about the hotspot-dev mailing list