Skip to content

Commit 489e3ff

Browse files
Define ECMULT_GEN_PREC_BITS
This is configurable in upstream now. Fixes #214. So far, this just sets it to `4`. It's tempting to set it to `2` when `lowmemory` is enabled but `lowmemory` is about RAM and not about binary size. This is better addressed by a resolution of #193.
1 parent 0782872 commit 489e3ff

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

secp256k1-sys/build.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,12 +52,13 @@ fn main() {
5252
.include("depend/secp256k1/src")
5353
.flag_if_supported("-Wno-unused-function") // some ecmult stuff is defined but not used upstream
5454
.define("SECP256K1_BUILD", Some("1"))
55+
.define("ENABLE_MODULE_ECDH", Some("1"))
56+
.define("ECMULT_GEN_PREC_BITS", Some("4"))
5557
// TODO these three should be changed to use libgmp, at least until secp PR 290 is merged
5658
.define("USE_NUM_NONE", Some("1"))
5759
.define("USE_FIELD_INV_BUILTIN", Some("1"))
58-
.define("USE_SCALAR_INV_BUILTIN", Some("1"))
59-
.define("ENABLE_MODULE_ECDH", Some("1"));
60-
60+
.define("USE_SCALAR_INV_BUILTIN", Some("1"));
61+
6162
if cfg!(feature = "lowmemory") {
6263
base_config.define("ECMULT_WINDOW_SIZE", Some("4")); // A low-enough value to consume neglible memory
6364
} else {

0 commit comments

Comments
 (0)