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

Fix instantiate_with_code benchmark #10146

Closed
wants to merge 3 commits into from
Closed

Fix instantiate_with_code benchmark #10146

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
93e7a93
Make sure that execution time isn't included in the instantiate_with_…
athei Nov 1, 2021
9e0c639
Merge branch 'master' of https://github.com/paritytech/substrate into…
Nov 2, 2021
fae5082
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
Nov 2, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion frame/contracts/src/benchmarking/code.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,7 @@ where
// and therefore grow in size when the contract grows. We are not allowed to overshoot
// because of the maximum code size that is enforced by `instantiate_with_code`.
let expansions = (target_bytes.saturating_sub(63) / 6).saturating_sub(1);
const EXPANSION: [Instruction; 4] = [I32Const(0), If(BlockType::NoResult), Return, End];
const EXPANSION: [Instruction; 4] = [I32Const(1), If(BlockType::NoResult), Return, End];
ModuleDefinition {
call_body: Some(body::repeated(expansions, &EXPANSION)),
memory: Some(ImportedMemory::max::<T>()),
Expand Down
Loading