RFR: 8303415: Add VM_Version::is_intrinsic_supported(id)

[Vladimir Kozlov]

On Thu, 9 Mar 2023 04:32:06 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> @dholmes-ora  I can remove parameter if I modify caller code like next:
>> 
>> -    is_available = compiler != NULL && compiler->is_intrinsic_available(mh, C->directive()) &&
>> -                   compiler->is_virtual_intrinsic_supported(id, is_virtual);
>> +    is_available = compiler != NULL && compiler->is_intrinsic_available(mh, C->directive());
>> +    if (is_available && is_virtual) {
>> +      is_available = compiler->is_virtual_intrinsic_supported(id);
>> +    }
>> 
>> Will it satisfy you?
>
> How many callers are there? From an API design perspective this method is either only for virtual intrinsics, so no parameter needed, or it is for general intrinsics and the parameter indicates what type.

This is the only place where it is called.

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

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