Skip to content

deduplication for COPY operations #1655

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.

Sign up for GitHub

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

Closed
harsh-ps-2003 wants to merge 9 commits into from
Closed

deduplication for COPY operations #1655

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
9d196f3
add copy_cost for *COPY operations
harsh-ps-2003 Jul 24, 2024
6b15eee
seperate common logic of opcodes to a method
harsh-ps-2003 Jul 25, 2024
1b72873
fmt nit
harsh-ps-2003 Jul 25, 2024
0453a50
performance nit
harsh-ps-2003 Jul 26, 2024
9c685ad
fmt nit
harsh-ps-2003 Jul 26, 2024
dbc2d57
test nits
harsh-ps-2003 Jul 26, 2024
1474059
fmt nit
harsh-ps-2003 Jul 26, 2024
a1a488b
refactor
harsh-ps-2003 Aug 6, 2024
f92ff49
Merge branch 'bluealloy:main' into copycode
harsh-ps-2003 Aug 6, 2024
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
Next Next commit
add copy_cost for *COPY operations
  • Loading branch information
@harsh-ps-2003
harsh-ps-2003 committed Jul 24, 2024
commit 9d196f33687491ed519e3493a51ae4c77e56f01f
9 changes: 7 additions & 2 deletions crates/interpreter/src/gas/calc.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ pub fn exp_cost(spec_id: SpecId, power: U256) -> Option<u64> {
/// `*COPY` opcodes cost calculation.
#[inline]
pub const fn verylowcopy_cost(len: u64) -> Option<u64> {
VERYLOW.checked_add(tri!(cost_per_word(len, COPY)))
copy_cost(VERYLOW, len, COPY)
}

/// `EXTCODECOPY` opcode cost calculation.
Expand All @@ -131,7 +131,12 @@ pub const fn extcodecopy_cost(spec_id: SpecId, len: u64, is_cold: bool) -> Optio
} else {
20
};
base_gas.checked_add(tri!(cost_per_word(len, COPY)))
copy_cost(base_gas, len, COPY)
}

#[inline]
pub const fn copy_cost(base_cost: u64, len: u64, copy_cost_per_word: u64) -> Option<u64> {
base_cost.checked_add(tri!(cost_per_word(len, copy_cost_per_word)))
}

/// `LOG` opcode cost calculation.
Expand Down