From 05d30fc7cf58ef57238295559aac709338e3c4ea Mon Sep 17 00:00:00 2001 From: chriseth Date: Thu, 3 Mar 2016 16:56:22 +0100 Subject: Add delegatecall to the optimizer. --- GasMeter.cpp | 12 ++++++++---- SemanticInformation.cpp | 3 +++ 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/GasMeter.cpp b/GasMeter.cpp index b792f04d..93583169 100644 --- a/GasMeter.cpp +++ b/GasMeter.cpp @@ -126,18 +126,22 @@ GasMeter::GasConsumption GasMeter::estimateMax(AssemblyItem const& _item) } case Instruction::CALL: case Instruction::CALLCODE: + case Instruction::DELEGATECALL: + { gas = m_schedule.callGas; if (u256 const* value = classes.knownConstant(m_state->relativeStackElement(0))) gas += (*value); else gas = GasConsumption::infinite(); - if (_item.instruction() != Instruction::CALLCODE) + if (_item.instruction() == Instruction::CALL) gas += m_schedule.callNewAccountGas; // We very rarely know whether the address exists. - if (!classes.knownZero(m_state->relativeStackElement(-2))) + int valueSize = _item.instruction() == Instruction::DELEGATECALL ? 0 : 1; + if (!classes.knownZero(m_state->relativeStackElement(-1 - valueSize))) gas += m_schedule.callValueTransferGas; - gas += memoryGas(-3, -4); - gas += memoryGas(-5, -6); + gas += memoryGas(-2 - valueSize, -3 - valueSize); + gas += memoryGas(-4 - valueSize, -5 - valueSize); break; + } case Instruction::CREATE: gas = m_schedule.createGas; gas += memoryGas(-1, -2); diff --git a/SemanticInformation.cpp b/SemanticInformation.cpp index 309bbe2b..ea579b83 100644 --- a/SemanticInformation.cpp +++ b/SemanticInformation.cpp @@ -134,6 +134,7 @@ bool SemanticInformation::isDeterministic(AssemblyItem const& _item) { case Instruction::CALL: case Instruction::CALLCODE: + case Instruction::DELEGATECALL: case Instruction::CREATE: case Instruction::GAS: case Instruction::PC: @@ -157,6 +158,7 @@ bool SemanticInformation::invalidatesMemory(Instruction _instruction) case Instruction::MSTORE8: case Instruction::CALL: case Instruction::CALLCODE: + case Instruction::DELEGATECALL: return true; default: return false; @@ -169,6 +171,7 @@ bool SemanticInformation::invalidatesStorage(Instruction _instruction) { case Instruction::CALL: case Instruction::CALLCODE: + case Instruction::DELEGATECALL: case Instruction::CREATE: case Instruction::SSTORE: return true; -- cgit