Skip to content

Commit d94a927

Browse files
committed
build: Adjust the default size of the precomputed table for signing
The new default value of the precomputed table for signing is aligned with Bitcoin Core's default. See: bitcoin/bitcoin@a057869
1 parent fcc5d73 commit d94a927

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ include(CheckStringOptionValue)
9090
check_string_option_value(SECP256K1_ECMULT_WINDOW_SIZE)
9191
add_compile_definitions(ECMULT_WINDOW_SIZE=${SECP256K1_ECMULT_WINDOW_SIZE})
9292

93-
set(SECP256K1_ECMULT_GEN_KB 22 CACHE STRING "The size of the precomputed table for signing in multiples of 1024 bytes (on typical platforms). Larger values result in possibly better signing or key generation performance at the cost of a larger table. Valid choices are 2, 22, 86. The default value is a reasonable setting for desktop machines (currently 22). [default=22]")
93+
set(SECP256K1_ECMULT_GEN_KB 86 CACHE STRING "The size of the precomputed table for signing in multiples of 1024 bytes (on typical platforms). Larger values result in possibly better signing or key generation performance at the cost of a larger table. Valid choices are 2, 22, 86. The default value is a reasonable setting for desktop machines (currently 86). [default=86]")
9494
set_property(CACHE SECP256K1_ECMULT_GEN_KB PROPERTY STRINGS 2 22 86)
9595
check_string_option_value(SECP256K1_ECMULT_GEN_KB)
9696
if(SECP256K1_ECMULT_GEN_KB EQUAL 2)

configure.ac

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -216,9 +216,9 @@ AC_ARG_WITH([ecmult-window], [AS_HELP_STRING([--with-ecmult-window=SIZE],
216216
AC_ARG_WITH([ecmult-gen-kb], [AS_HELP_STRING([--with-ecmult-gen-kb=2|22|86],
217217
[The size of the precomputed table for signing in multiples of 1024 bytes (on typical platforms).]
218218
[Larger values result in possibly better signing/keygeneration performance at the cost of a larger table.]
219-
[The default value is a reasonable setting for desktop machines (currently 22). [default=22]]
219+
[The default value is a reasonable setting for desktop machines (currently 86). [default=86]]
220220
)],
221-
[set_ecmult_gen_kb=$withval], [set_ecmult_gen_kb=22])
221+
[set_ecmult_gen_kb=$withval], [set_ecmult_gen_kb=86])
222222

223223
AC_ARG_WITH([valgrind], [AS_HELP_STRING([--with-valgrind=yes|no|auto],
224224
[Build with extra checks for running inside Valgrind [default=auto]]

0 commit comments

Comments
 (0)