msvc: Cleanup after upgrading libsecp256k1 up to 0.3.0#27580
msvc: Cleanup after upgrading libsecp256k1 up to 0.3.0#27580fanquake merged 1 commit intobitcoin:masterfrom
Conversation
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
|
ACK |
|
Actually I still get the untracked files warning after merge -- am i doing something wrong? Or would it make any sense to add these to gitignore? |
Try to clean your local repo first. For example, |
libsecp256k1 v0.3.0:
This PR removed the code that has been unused since #27230.
The
USE_ASM_X86_64is now undefined explicitly (but actually it seems a bit redundant).The
ECMULT_GEN_PREC_BITSandECMULT_WINDOW_SIZEmacros are defined by the source code to their defaults.Considering the upcoming CMake-based build system, these changes have a low-priority.