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

[yul-phaser] Make Program noncopyable

Browse Source 
- Copying worked but resulted in OptimiserStepContext having a reference to NameDispenser instance of the other object.
This commit is contained in:
cameel 2020-01-22 19:38:19 +01:00
parent 14d726ff01
commit 2aa42b32e5
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tools/yulPhaser/Program.h
View File

@ -21,6 +21,8 @@
#include <libyul/optimiser/NameDispenser.h> #include <libyul/optimiser/NameDispenser.h>
#include <libyul/AsmData.h> #include <libyul/AsmData.h>
#include <boost/noncopyable.hpp>
#include <optional> #include <optional>
#include <set> #include <set>
#include <string> #include <string>
@ -54,9 +56,17 @@ namespace solidity::phaser
* The class allows the user to apply extra optimisations and obtain metrics and general * The class allows the user to apply extra optimisations and obtain metrics and general
* information about the resulting syntax tree. * information about the resulting syntax tree.
*/ */
class Program class Program: private boost::noncopyable
{ {
public: public:
Program(Program&& program):
m_ast(std::move(program.m_ast)),
m_nameDispenser(std::move(program.m_nameDispenser)),
// Creating a new instance because a copied one would have a dangling reference to the old dispenser
m_optimiserStepContext{program.m_optimiserStepContext.dialect, m_nameDispenser, s_externallyUsedIdentifiers}
{}
Program operator=(Program&& program) = delete;
static Program load(std::string const& _sourcePath); static Program load(std::string const& _sourcePath);
void optimise(std::vector<std::string> const& _optimisationSteps); void optimise(std::vector<std::string> const& _optimisationSteps);