Skip to content
This repository was archived by the owner on Oct 4, 2019. It is now read-only.

EIP-150 for Ethereum Classic cpp-ethereum. #10

Open
marcusrbrown wants to merge 23 commits into
base: develop
Choose a base branch
from
Open

EIP-150 for Ethereum Classic cpp-ethereum. #10

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
1c609fd
EIP150 version1 gasPrice change proposal
Oct 10, 2016
41d22dc
update gas price for
Oct 11, 2016
9eaaad0
fixing PR
Oct 11, 2016
975f09a
Implement EIP150 1b for calls
chfast Oct 12, 2016
e78f769
Implement EIP150 1b (63/64 gas) for CREATE
chfast Oct 12, 2016
30a7f20
Refactor EIP150 1b changes for improve readability
chfast Oct 12, 2016
30a38c4
Additional cost of SELFDESTRUCT sending to non-existing account
chfast Oct 12, 2016
3250d78
update EIP150 test filler
Oct 12, 2016
1cbb407
Improve EVMSchedule
chfast Oct 12, 2016
f7c82e5
Rename EIP150 related objects
chfast Oct 12, 2016
f3ed580
test: add gasCheck on creation of a contract
Oct 12, 2016
394cf2c
Rework code for "all but one 64th" rule. NFC
chfast Oct 13, 2016
2adbdc0
EIP150 MemExpandingTests
Oct 13, 2016
ed38acb
more EIP150 state tests
Oct 14, 2016
f013d6b
change frontierCompatibilityModeLimit to homsteadForkBlock
Oct 14, 2016
1a97c6a
Restore stack limit of 1024 for EIP150
chfast Oct 14, 2016
dc81af7
Adapt Homestead tests to EIP150 specs
Oct 14, 2016
375ac97
fix incorrect bnumber in state tests
Oct 14, 2016
ea24960
Move blockchain tests to be run on EIP150 rules
Oct 14, 2016
255c249
update blockchainEIP fillers
Oct 15, 2016
9612186
blockhain tests that crash cpp eth
Oct 15, 2016
2d6fc35
change EIP150 bnumber to 2463000
Oct 17, 2016
7c1d277
Change the EIP-150 block number from 2463000 to 2500000 for Ethereum …
marcusrbrown Oct 17, 2016
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
17 changes: 11 additions & 6 deletions libevm/VMCalls.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,15 +153,20 @@ bool VM::caseCallSetup(CallParameters *callParams)
updateIOGas();

// "Static" costs already applied. Calculate call gas.
u256 requestedCallGas = *m_sp;
u256 maxAllowedCallGas = *m_io_gas - *m_io_gas / 64;
u256 callGas = m_schedule->staticCallDepthLimit() ? requestedCallGas :
std::min(requestedCallGas, maxAllowedCallGas);
m_runGas = toUint64(callGas);
if (m_schedule->staticCallDepthLimit())
// With static call depth limit we just charge the provided gas amount.
callParams->gas = *m_sp;
else
{
// Apply "all but one 64th" rule.
u256 maxAllowedCallGas = *m_io_gas - *m_io_gas / 64;
callParams->gas = std::min(*m_sp, maxAllowedCallGas);
}

Copy link
Member

@elaineo elaineo Oct 21, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the "all but one 64th" rule?

m_runGas = toUint64(callParams->gas);
onOperation();
updateIOGas();

callParams->gas = callGas;
if (m_op != Instruction::DELEGATECALL && *(m_sp - 2) > 0)
callParams->gas += m_schedule->callStipend;
--m_sp;
Expand Down