Use "in" tag for jumps from unpacker to function.

This commit is contained in:
chriseth 2018-11-13 10:53:40 +01:00 committed by Leonardo Alt
parent 96333f3033
commit 0e08fb7469
2 changed files with 8 additions and 2 deletions

View File

@ -167,7 +167,10 @@ public:
/// the data.
CompilerContext& appendConditionalRevert(bool _forwardReturnData = false);
/// Appends a JUMP to a specific tag
CompilerContext& appendJumpTo(eth::AssemblyItem const& _tag) { m_asm->appendJump(_tag); return *this; }
CompilerContext& appendJumpTo(
eth::AssemblyItem const& _tag,
eth::AssemblyItem::JumpType _jumpType = eth::AssemblyItem::JumpType::Ordinary
) { *m_asm << _tag.pushTag(); return appendJump(_jumpType); }
/// Appends pushing of a new tag and @returns the new tag.
eth::AssemblyItem pushNewTag() { return m_asm->append(m_asm->newPushTag()).tag(); }
/// @returns a new tag without pushing any opcodes or data

View File

@ -344,7 +344,10 @@ void ContractCompiler::appendFunctionSelector(ContractDefinition const& _contrac
m_context << Instruction::DUP1 << Instruction::CALLDATASIZE << Instruction::SUB;
CompilerUtils(m_context).abiDecode(functionType->parameterTypes());
}
m_context.appendJumpTo(m_context.functionEntryLabel(functionType->declaration()));
m_context.appendJumpTo(
m_context.functionEntryLabel(functionType->declaration()),
eth::AssemblyItem::JumpType::IntoFunction
);
m_context << returnTag;
// Return tag and input parameters get consumed.
m_context.adjustStackOffset(