RFR: 8369685: RISC-V: refactor code related to RVFeatureValue::enabled [v5]
Fei Yang
fyang at openjdk.org
Fri Oct 17 10:34:03 UTC 2025
On Fri, 17 Oct 2025 08:54:21 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:
>
> review
Latest version looks good. Thanks for the update.
-------------
Marked as reviewed by fyang (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/27771#pullrequestreview-3349374341
More information about the hotspot-dev
mailing list