RFR: 8313372: [JVMCI] Export vmIntrinsics::is_intrinsic_available results to JVMCI compilers. [v2]
Vladimir Kozlov
kvn at openjdk.org
Sat Aug 12 00:46:30 UTC 2023
On Thu, 10 Aug 2023 14:28:39 GMT, Yudi Zheng <yzheng at openjdk.org> wrote:
>> I don't having the same logic in two places, because then those two places need to be kept in sync. Either the stubs should be generated based on is_intrinsic_supported(), or is_intrinsic_supported() should check if the stub was generated.
>
>> I don't having the same logic in two places, because then those two places need to be kept in sync. Either the stubs should be generated based on is_intrinsic_supported(), or is_intrinsic_supported() should check if the stub was generated.
>
> I have dropped the redundant CPU feature checks, and for those intrinsics with stubs, I tested if the stub pointer is nullptr in `C2Compiler::is_intrinsic_supported`
@mur47x111 please, rerun testing with latest version before integration.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15133#issuecomment-1675567914
More information about the graal-dev
mailing list