-
Notifications
You must be signed in to change notification settings - Fork 5.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EVM version #3569
EVM version #3569
Changes from all commits
5ab4a1a
8578571
f75a411
60f7be5
83515ea
f2f61f1
a53d6b4
dc317a4
739533e
05cc5f2
1e26011
7b0272c
982476f
7171ac0
4ce0e77
1246917
7f8e573
5a54cd5
6ec4517
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -353,7 +353,7 @@ void Assembly::injectStart(AssemblyItem const& _i) | |
m_items.insert(m_items.begin(), _i); | ||
} | ||
|
||
Assembly& Assembly::optimise(bool _enable, bool _isCreation, size_t _runs) | ||
Assembly& Assembly::optimise(bool _enable, EVMVersion _evmVersion, bool _isCreation, size_t _runs) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Woohoo, I should be using this for the bitwise shifting optimiser rules in #3580. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, it was a really painful process. This setting is needed literally everywhere... |
||
{ | ||
OptimiserSettings settings; | ||
settings.isCreation = _isCreation; | ||
|
@@ -365,6 +365,7 @@ Assembly& Assembly::optimise(bool _enable, bool _isCreation, size_t _runs) | |
settings.runCSE = true; | ||
settings.runConstantOptimiser = true; | ||
} | ||
settings.evmVersion = _evmVersion; | ||
settings.expectedExecutionsPerDeployment = _runs; | ||
optimise(settings); | ||
return *this; | ||
|
@@ -482,6 +483,7 @@ map<u256, u256> Assembly::optimiseInternal( | |
ConstantOptimisationMethod::optimiseConstants( | ||
_settings.isCreation, | ||
_settings.isCreation ? 1 : _settings.expectedExecutionsPerDeployment, | ||
_settings.evmVersion, | ||
*this, | ||
m_items | ||
); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why were these removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we don't have any submodules anymore, do we? I can remove it from this PR.