cerc-io/solidity
10
0
Fork 0
mirror of https://github.com/ethereum/solidity synced 2023-10-03 13:03:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #4166 from alibabe/fix-3718-ret

Browse Source 
Fix the Heisenbug
This commit is contained in:
chriseth 2018-07-16 13:38:42 +02:00 committed by GitHub
commit 236284a264
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
solc/CommandLineInterface.cpp
View File

@ -986,10 +986,15 @@ void CommandLineInterface::handleAst(string const& _argStr)
map<ASTNode const*, eth::GasMeter::GasConsumption> gasCosts; map<ASTNode const*, eth::GasMeter::GasConsumption> gasCosts;
// FIXME: shouldn't this be done for every contract? // FIXME: shouldn't this be done for every contract?
if (m_compiler->runtimeAssemblyItems(m_compiler->lastContractName())) if (m_compiler->runtimeAssemblyItems(m_compiler->lastContractName()))
gasCosts = GasEstimator::breakToStatementLevel( {
//NOTE: keep the local variable `ret` to prevent a Heisenbug that could happen on certain mac os platform.
//See: https://github.com/ethereum/solidity/issues/3718 for details.
auto ret = GasEstimator::breakToStatementLevel(
GasEstimator(m_evmVersion).structuralEstimation(*m_compiler->runtimeAssemblyItems(m_compiler->lastContractName()), asts), GasEstimator(m_evmVersion).structuralEstimation(*m_compiler->runtimeAssemblyItems(m_compiler->lastContractName()), asts),
asts asts
); );
gasCosts = ret;
}
bool legacyFormat = !m_args.count(g_argAstCompactJson); bool legacyFormat = !m_args.count(g_argAstCompactJson);
if (m_args.count(g_argOutputDir)) if (m_args.count(g_argOutputDir))