Skip to content
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

Add some manifest sbor invocations #1688

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

0xOmarA
Copy link
Member

@0xOmarA 0xOmarA commented Jan 19, 2024

Summary

A small PR that adds a number of ManifestSbor invocations that the toolkit needs.

Copy link

Benchmark for a568741

Click to view benchmark
Test Base PR %
costing::bench_prepare_wasm 62.8±0.51ms 62.7±0.57ms -0.16%
costing::decode_sbor 12.9±0.02µs 13.2±0.06µs +2.33%
costing::decode_sbor_bytes 38.0±0.04µs 35.5±0.05µs -6.58%
costing::deserialize_wasm 1232.1±2.85µs 1233.8±2.35µs +0.14%
costing::instantiate_flash_loan 3.8±0.51ms 4.0±0.57ms +5.26%
costing::instantiate_radiswap 5.0±0.05ms 5.0±0.05ms 0.00%
costing::spin_loop 21.3±0.01ms 21.4±0.02ms +0.47%
costing::validate_sbor_payload 26.4±0.02µs 27.0±0.02µs +2.27%
costing::validate_sbor_payload_bytes 353.7±0.37ns 355.3±0.33ns +0.45%
costing::validate_secp256k1 80.3±0.11µs 80.2±0.06µs -0.12%
costing::validate_wasm 34.3±0.04ms 34.6±0.03ms +0.87%
decimal::add/0 7.2±0.00ns 7.2±0.00ns 0.00%
decimal::add/rust-native 9.5±0.01ns 9.5±0.00ns 0.00%
decimal::add/wasmer 134.4±0.13ns 133.8±0.13ns -0.45%
decimal::add/wasmer-call-native 518.3±0.18ns 519.3±0.38ns +0.19%
decimal::add/wasmi 458.1±0.16ns 448.2±0.12ns -2.16%
decimal::add/wasmi-call-native 3.3±0.01µs 3.5±0.03µs +6.06%
decimal::div/0 166.1±0.07ns 166.0±0.15ns -0.06%
decimal::from_string/0 153.0±0.08ns 153.0±0.15ns 0.00%
decimal::mul/0 129.8±0.08ns 129.8±0.08ns 0.00%
decimal::mul/rust-native 132.5±0.16ns 132.6±0.15ns +0.08%
decimal::mul/wasmer 1719.9±0.77ns 1729.8±0.59ns +0.58%
decimal::mul/wasmer-call-native 655.7±0.32ns 660.5±0.10ns +0.73%
decimal::mul/wasmi 26.8±0.03µs 26.0±0.03µs -2.99%
decimal::mul/wasmi-call-native 3.4±0.01µs 3.4±0.03µs 0.00%
decimal::pow/0 621.6±0.22ns 621.8±0.28ns +0.03%
decimal::pow/rust-native 609.9±0.23ns 605.1±0.16ns -0.79%
decimal::pow/wasmer 7.5±0.00µs 7.5±0.00µs 0.00%
decimal::pow/wasmer-call-native 1076.5±0.25ns 1075.8±0.30ns -0.07%
decimal::pow/wasmi 126.4±0.21µs 125.0±0.08µs -1.11%
decimal::pow/wasmi-call-native 3.4±0.01µs 3.4±0.01µs 0.00%
decimal::root/0 9.1±0.00µs 9.3±0.01µs +2.20%
decimal::sub/0 7.2±0.01ns 7.2±0.00ns 0.00%
decimal::to_string/0 488.7±0.17ns 492.0±0.35ns +0.68%
precise_decimal::add/0 8.7±0.00ns 8.7±0.00ns 0.00%
precise_decimal::add/rust-native 10.4±0.02ns 10.4±0.02ns 0.00%
precise_decimal::add/wasmer 140.5±0.04ns 140.4±0.04ns -0.07%
precise_decimal::add/wasmer-call-native 533.2±0.21ns 536.1±0.18ns +0.54%
precise_decimal::add/wasmi 528.9±0.10ns 529.9±2.56ns +0.19%
precise_decimal::add/wasmi-call-native 3.5±0.02µs 3.7±0.03µs +5.71%
precise_decimal::div/0 261.4±0.07ns 261.3±0.08ns -0.04%
precise_decimal::from_string/0 196.5±0.46ns 196.7±0.25ns +0.10%
precise_decimal::mul/0 278.7±0.10ns 278.7±0.07ns 0.00%
precise_decimal::mul/rust-native 259.5±0.12ns 259.7±0.14ns +0.08%
precise_decimal::mul/wasmer 4.0±0.00µs 4.0±0.00µs 0.00%
precise_decimal::mul/wasmer-call-native 831.4±0.23ns 827.0±0.34ns -0.53%
precise_decimal::mul/wasmi 74.6±0.03µs 73.4±0.05µs -1.61%
precise_decimal::mul/wasmi-call-native 3.9±0.01µs 3.9±0.02µs 0.00%
precise_decimal::pow/0 1573.9±0.27ns 1575.0±1.23ns +0.07%
precise_decimal::pow/rust-native 1274.2±2.00ns 1273.4±0.31ns -0.06%
precise_decimal::pow/wasmer 19.2±0.01µs 19.2±0.00µs 0.00%
precise_decimal::pow/wasmer-call-native 2.0±0.00µs 2.0±0.00µs 0.00%
precise_decimal::pow/wasmi 358.0±0.10µs 355.5±0.26µs -0.70%
precise_decimal::pow/wasmi-call-native 7.7±0.03µs 7.6±0.03µs -1.30%
precise_decimal::root/0 62.4±0.02µs 61.4±0.02µs -1.60%
precise_decimal::sub/0 8.9±0.01ns 8.9±0.00ns 0.00%
precise_decimal::to_string/0 744.6±0.23ns 744.9±0.10ns +0.04%
schema::validate_payload 325.1±0.24µs 323.0±0.39µs -0.65%
transaction::radiswap 5.2±0.04ms 5.2±0.04ms 0.00%
transaction::transfer 1677.8±2.54µs 1687.0±2.68µs +0.55%
transaction_processing::prepare 2.4±0.00ms 2.4±0.00ms 0.00%
transaction_processing::prepare_and_decompile 6.2±0.01ms 6.2±0.01ms 0.00%
transaction_processing::prepare_and_decompile_and_recompile 22.3±0.08ms 22.7±0.10ms +1.79%
transaction_validation::validate_manifest 43.8±0.03µs 43.8±0.07µs 0.00%
transaction_validation::verify_bls_2KB 999.2±2.68µs 1015.1±8.24µs +1.59%
transaction_validation::verify_bls_32B 1003.7±16.99µs 1010.8±17.94µs +0.71%
transaction_validation::verify_ecdsa 78.0±0.06µs 78.0±0.07µs 0.00%
transaction_validation::verify_ed25519 51.8±0.08µs 51.5±0.69µs -0.58%

@0xOmarA 0xOmarA merged commit 542176d into release/anemone Jan 19, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants