Fix stack compressor wrt literal arguments.

This commit is contained in:
chriseth 2020-07-23 13:55:17 +02:00
parent fc34c81f4c
commit b1f79072df

View File

@ -163,14 +163,21 @@ NoOutputEVMDialect::NoOutputEVMDialect(EVMDialect const& _copyFrom):
{ {
for (auto& fun: m_functions) for (auto& fun: m_functions)
{ {
size_t parameters = fun.second.parameters.size();
size_t returns = fun.second.returns.size(); size_t returns = fun.second.returns.size();
fun.second.generateCode = [=](FunctionCall const& _call, AbstractAssembly& _assembly, BuiltinContext&, std::function<void(Expression const&)> _visitExpression) fun.second.generateCode = [=](FunctionCall const& _call, AbstractAssembly& _assembly, BuiltinContext&, std::function<void(Expression const&)> _visitExpression)
{ {
for (auto const& arg: _call.arguments | boost::adaptors::reversed) size_t visited = 0;
_visitExpression(arg); for (size_t j = 0; j < _call.arguments.size(); j++)
{
size_t const i = _call.arguments.size() - j - 1;
if (!(fun.second.literalArguments && (*fun.second.literalArguments)[i]))
{
_visitExpression(_call.arguments[i]);
visited++;
}
}
for (size_t i = 0; i < parameters; i++) for (size_t i = 0; i < visited; i++)
_assembly.appendInstruction(evmasm::Instruction::POP); _assembly.appendInstruction(evmasm::Instruction::POP);
for (size_t i = 0; i < returns; i++) for (size_t i = 0; i < returns; i++)