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