On Sun, 9 Aug 2026 01:34:27 GMT, Shawn Emery <[email protected]> wrote:

>> This fix allows for more accurate comparisons between intrinsics and Java 
>> performance for P256 conditional set operations by emulating production 
>> lookup EC operations.  Before this fix, benchmarks showed a 53% regression 
>> for intrinsics vs. Java due to constant propagation and dead-code removal by 
>> C2.  After this fix, a 52% performance increase is observed with intrinsics 
>> vs. Java.
>> 
>> ---------
>> - [X] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Shawn Emery has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Emulate production operations via EC lookups

test/micro/org/openjdk/bench/javax/crypto/full/PolynomialP256Bench.java line 98:

> 96:         MutableIntegerModuloP test2 = one.mutable();
> 97:         for (int i = 0; i< 10000; i++) {
> 98:             test1.conditionalSet(test2, 0);

Can you simply disallow inlining of `conditionalSet` instead?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32047#discussion_r3836581835

Reply via email to