-
Notifications
You must be signed in to change notification settings - Fork 102
perf(levm): use malachite for modexp #4045
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
Conversation
Lines of code reportTotal lines added: Detailed view
|
Benchmark Results ComparisonNo significant difference was registered for any benchmark run. Detailed ResultsBenchmark Results: BubbleSort
Benchmark Results: ERC20Approval
Benchmark Results: ERC20Mint
Benchmark Results: ERC20Transfer
Benchmark Results: Factorial
Benchmark Results: FactorialRecursive
Benchmark Results: Fibonacci
Benchmark Results: FibonacciRecursive
Benchmark Results: ManyHashes
Benchmark Results: MstoreBench
Benchmark Results: Push
Benchmark Results: SstoreBench_no_opt
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM great PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The "EVM" aspect of it is perfectly fine. Maybe someone with more math knowledge would be able to give a deeper review in terms of the precompile itself.
Benchmark Block Execution Results Comparison Against Main
|
Use malachite library to improve the performance of the modexp precompile. There are speedups up to 7-8x observed in the new benchmarks. --------- Co-authored-by: Tomás Arjovsky <[email protected]>
Use malachite library to improve the performance of the modexp precompile. There are speedups up to 7-8x observed in the new benchmarks.