RFR: 8369685: RISC-V: refactor code related to RVFeatureValue::enabled [v3]
Robbin Ehn
rehn at openjdk.org
Thu Oct 16 15:40:09 UTC 2025
On Thu, 16 Oct 2025 14:03:10 GMT, Hamlin Li <mli at openjdk.org> wrote:
>> 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:
>
> use DEFAULT_VALUE for RVExtFeatureValue::value()
Marked as reviewed by rehn (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/27771#pullrequestreview-3345599056
More information about the hotspot-dev
mailing list