ethereumjs-monorepo icon indicating copy to clipboard operation
ethereumjs-monorepo copied to clipboard

BLS: use noble-curves MSM

Open paulmillr opened this issue 1 year ago • 4 comments

It has been implemented here https://github.com/paulmillr/noble-curves?tab=readme-ov-file#multi-scalar-multiplication

It would appear in v1.6.0 which is yet to be released, but could be tested right away.

paulmillr avatar Sep 02 '24 15:09 paulmillr

Just to check, we would use this method for msmG1 and msmG2 (with their respective methods) in evm/src/precompiles/bls12_381/noble.ts right? Since it currently does "just multiply" the array, and not this multi-multiplication.

jochem-brouwer avatar Sep 02 '24 16:09 jochem-brouwer

Yes

paulmillr avatar Sep 02 '24 16:09 paulmillr

Just to note: v1.6 has now been released, so this issue can be tackled. Also to note, that this has not necessarily to be done by the team members involved with the BLS code, since this is a very local change.

In fact it might be a nice occasion to first-round engage with the BLS precompile code in a small way!

holgerd77 avatar Sep 20 '24 08:09 holgerd77

Test runs for this can be taken from #3471 (ethereum-tests test runs a lot more relevant then our own tests, these are basically doing nothing except stating that the precompile is there at all 😂 )

holgerd77 avatar Sep 25 '24 13:09 holgerd77