-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated fork to use v0.3.11 & made points public #1
Conversation
Commentary is reformatted as a result of goimports' update.
The typo is benign because vec384 and vec256 are indistinguishable when used as arguments to a function, both are pointers to the first limb.
Unlike gcc, clang refuses to compile vec_nbits() as constexpr. Hence this breaking change.
Trouble is that cc-rs fails to pass proper --target to the C compiler, unless it's specifically clang.
vec_cswap is not actually used, or rather the macros that reference it are not used, so it's a just-in-case fix.
Add from(const uint32_t a[2*n]) method that yields reduced result in representation suitable for usage as scalar in EC multiplication.
SWIG-based bindings require only SWIG-specific symbols to be accesible in corresponding shared libraries.
Thanks to Yunjong Jeong (@blukat29) for report.
I think I found a way to make the msm api work against the regular blst fork: @ToufeeqP See the logic put together in the Not sure where else you're using this, but it might be helpful to pull in that file (or just use the |
Hey @Aphoh, thanks for looking into it! I believe this is also being used in avail-core, but we specifically needed blst version 0.3.11 due to deps in the bridge pallet. However, we’ve decided not to use that library directly in the pallet for now, so I’ll go ahead and close this PR. If we need it in the future, I can reopen it |
this commit has our changes