Page MenuHomePhabricator

parameterize ecmult_const over input size
ClosedPublic

Authored by deadalnix on Sat, Jan 11, 13:39.

Details

Reviewers
markblundeberg
Group Reviewers
Restricted Project
Commits
rABC95fb86547648: parameterize ecmult_const over input size
Summary

This is a backport of secp56k1's PR522

Test Plan
ninja check-secp256k1 bench-secp256k1

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

deadalnix created this revision.Sat, Jan 11, 13:39
Herald added a reviewer: Restricted Project. · View Herald TranscriptSat, Jan 11, 13:39
markblundeberg requested changes to this revision.Sun, Jan 12, 08:29
markblundeberg added a subscriber: markblundeberg.

link to PR please

This revision now requires changes to proceed.Sun, Jan 12, 08:29

Out of curiosity, why are we backporting these when Core doesn't have them in their master?

Out of curiosity, why are we backporting these when Core doesn't have them in their master?

I say why not :) We don't want to have a giant merge from hell at some point going forward.

markblundeberg accepted this revision.Mon, Jan 13, 23:16

Never mind, I was wrong -- Core does have these changes.

This revision is now accepted and ready to land.Mon, Jan 13, 23:16
This revision was automatically updated to reflect the committed changes.