cerc-io/solidity
15
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

OptimiserSuite: Remove VarNameCleaner from the list of available steps

Browse Source
This commit is contained in:
cameel 2020-01-29 18:05:18 +01:00 committed by chriseth
parent e41155cf48
commit 2ee7e6042b
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
libyul/optimiser/Suite.cpp
View File

@ -308,9 +308,7 @@ void OptimiserSuite::run(
if (ast.statements.size() > 1 && std::get<Block>(ast.statements.front()).statements.empty()) if (ast.statements.size() > 1 && std::get<Block>(ast.statements.front()).statements.empty())
ast.statements.erase(ast.statements.begin()); ast.statements.erase(ast.statements.begin());
} }
suite.runSequence({ VarNameCleaner::run(suite.m_context, ast);
VarNameCleaner::name
}, ast);
*_object.analysisInfo = AsmAnalyzer::analyzeStrictAssertCorrect(_dialect, _object); *_object.analysisInfo = AsmAnalyzer::analyzeStrictAssertCorrect(_dialect, _object);
} }
@ -366,9 +364,9 @@ map<string, unique_ptr<OptimiserStep>> const& OptimiserSuite::allSteps()
SSATransform, SSATransform,
StructuralSimplifier, StructuralSimplifier,
UnusedPruner, UnusedPruner,
VarDeclInitializer, VarDeclInitializer
VarNameCleaner
>(); >();
// Does not include VarNameCleaner because it destroys the property of unique names.
return instance; return instance;
} }

4
libyul/optimiser/VarNameCleaner.h
View File

@ -39,6 +39,10 @@ struct Dialect;
* renumbered by their base name. * renumbered by their base name.
* Function names are not modified. * Function names are not modified.
* *
* NOTE: This step destroys the promise of the Disambiguator and thus cannot
* be used in the main loop of the optimizer without running the disambiguator again.
* Because of that, it is not included in the step list of the Optimizer Suite.
*
* Prerequisites: Disambiguator, FunctionHoister, FunctionGrouper * Prerequisites: Disambiguator, FunctionHoister, FunctionGrouper
*/ */
class VarNameCleaner: public ASTModifier class VarNameCleaner: public ASTModifier