RFR: 8329538: Accelerate P256 on x86_64 using Montgomery intrinsic [v7]
Volodymyr Paprotski
duke at openjdk.org
Fri May 10 00:15:40 UTC 2024
On Thu, 9 May 2024 23:36:03 GMT, Anthony Scarpino <ascarpino at openjdk.org> wrote:
>> Volodymyr Paprotski has updated the pull request incrementally with one additional commit since the last revision:
>>
>> whitespace
>
> src/java.base/share/classes/sun/security/ec/ECOperations.java line 701:
>
>> 699: if (!m.equals(v)) {
>> 700: java.util.HexFormat hex = java.util.HexFormat.of();
>> 701: throw new RuntimeException();
>
> I think your cleanup went to far. You should have some message saying they are not equal and if you don't want to print hex, remove getting an instance.
I put the message back.. I removed it 'half'-intentionally; Was comparing against the original version and it didn't have any details, thought maybe should follow suit. But I did find this message helpful, so its back.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18583#discussion_r1596116606
More information about the build-dev
mailing list