Skip to content
This repository has been archived by the owner on Nov 18, 2022. It is now read-only.

Refactor initialization of data buffers #74

Merged
merged 8 commits into from
Aug 10, 2021
Merged

Refactor initialization of data buffers #74

merged 8 commits into from
Aug 10, 2021

Conversation

Flouse
Copy link
Collaborator

@Flouse Flouse commented Aug 6, 2021

  • refactor fuzzing
  • All the elements of global (static) arrays are initialized to default
    values (zero for arithmetic types and NULL for pointers).
    memset caused by the initialization of such arrays is not necessary.
    • perf: init debug_print_buffer in run_polyjuice() - da50d22
    • perf: init evm_memory before handle_message - e0cd641

- deprecate polyjuice-tests/fuzz/mock_generator_utils.h and follow the newest version of deps/godwoken-scripts/c/generator_utils.h
- mock_mint_sudt for new account
… unsigned char const*, unsigned int, unsigned char*)'
All the elements of global arrays are initialized to default values
(zero for arithmetic types and NULL for pointers).
`memset` caused by the initialization of such arrays is not necessary.
All the elements of global (static) arrays are initialized to default
values (zero for arithmetic types and NULL for pointers).
`memset` caused by the initialization of such arrays is not necessary.
@Flouse Flouse changed the title Perf Refactor initialization of data buffers Aug 6, 2021
@Flouse
Copy link
Collaborator Author

Flouse commented Aug 9, 2021

Deploy SimpleWallet Contract Testcase

Commit All Samples of Flame Graph Flame Graph
before perf 2e3e1a4 24.7 M Deploy_SimpleWallet.svg.txt
perf: remove some helpless debug_print_data #71 7.8 M pprof-deploy-wallet-0805.svg.txt
Refactor initialization of data buffers #74 2.0 M simple-wallet-0809.tar.gz

simple-wallet-0809.svg

image

c/polyjuice_utils.h Outdated Show resolved Hide resolved
Flouse added a commit to Flouse/godwoken-polyjuice that referenced this pull request Aug 9, 2021
@Flouse Flouse requested review from TheWaWaR, jjyr and xxuejie August 9, 2021 03:58
@Flouse Flouse merged commit 2800e89 into main Aug 10, 2021
@Flouse Flouse deleted the perf branch August 10, 2021 02:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants