RFR: 8354471: Assertion failure with -XX:-EnableX86ECoreOpts [v2]

[Sandhya Viswanathan]

On Tue, 15 Apr 2025 15:48:02 GMT, Volodymyr Paprotski <vpaprotski at openjdk.org> wrote:

>> The check to choose between AVX2 and AVX512 implementation was relying on `EnableX86ECoreOpts`. It should be relying on `supports_avxifma` and mirror the `UseIntPolyIntrinsics` check in `vm_version_x86.cpp`.
>> 
>> Note, in `stubGenerator_x86_64.cpp`, entry to the patched function is protected already:
>> 
>>   if (UseIntPolyIntrinsics) {
>>     StubRoutines::_intpoly_montgomeryMult_P256 = generate_intpoly_montgomeryMult_P256();
>>     StubRoutines::_intpoly_assign = generate_intpoly_assign();
>>   }
>
> Volodymyr Paprotski has updated the pull request incrementally with one additional commit since the last revision:
> 
>   flip the direction of if to prefer AVX512

Looks good to me.

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

Marked as reviewed by sviswanathan (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/24644#pullrequestreview-2769942677