Jonas Nick
Jonas Nick
Thanks, I think all the corrections were either addressed or pointed out in the PR to upstream libsecp256k1.
I think it's best if we merge the PR in upstream libsecp first and then merge the changes to this repo.
No one is working on this as far as I know.
> I hope I didn’t step on your toes with that. Definitely not. The approaches in both PRs seem to be very different. The goal of this PR is to...
Closing because I prefer #1734: - parallelism and the resulting speedup - much more extensible, while only adding a reasonable number of locs - easier to add and change granularity...
PR #1211 is an attempt to implement this.
clnrest has been merged in #688
Concept ACK on investigating fiat-crypto and (longer term) cryptopt. Would be great to have a verified implementation that's only slightly slower or even faster. Thanks @dderjoel for the benchmarks.
ACK ddf5b25 Same fix is also in #1544 (do you want to close that @real-or-random?).
Superseded by #1654