On Sun, 1 Mar 2026 14:30:47 GMT, Chen Liang <[email protected]> wrote:

>> Shawn Emery has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Remove IntrinsicCandidate for mult() method
>
> src/java.base/share/classes/sun/security/util/math/intpoly/IntegerPolynomial25519.java
>  line 155:
> 
>> 153: 
>> 154:     @Override
>> 155:     @IntrinsicCandidate
> 
> Is this intrinsic annotation incorrectly copied from 
> MontgomeryIntegerPolynomialP256.java? I only find an intrinsic of this name 
> and signature for that class.
> 
> The classFileParser intrinsic check doesn't result in VM crash for product 
> builds so you might have ignored its output message.

Thank you for catching this.  The method is currently not intrinsified but will 
be in the future (i.e., JDK-8371305).  Removing for now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29981#discussion_r2869596024

Reply via email to