RFR: 8369685: RISC-V: refactor code related to RVFeatureValue::enabled [v5]
Hamlin Li
mli at openjdk.org
Fri Oct 17 08:54:23 UTC 2025
On Fri, 17 Oct 2025 08:44:12 GMT, Robbin Ehn <rehn at openjdk.org> wrote:
>> Good idea. Maybe put them into `update_flag` instead? Or maybe a new method called `log_enabled` in `RVExtFeatures`? I prefer the latter.
>>
>> @robehn how do you think about it? That means `value()` is only a method in `RVNonExtFeatureValue`.
>
> Yes, that works.
@robehn Thanks for the quick response!
@RealFYang I choose to add a new method `log_enabled`, let me know how you think about it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27771#discussion_r2438905393
More information about the hotspot-dev
mailing list