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

Merge pull request #5221 from ethereum/splitRenamingLeftovers

Browse Source 
Some renaming leftovers.
This commit is contained in:
chriseth 2018-10-15 14:22:17 +02:00 committed by GitHub
commit 6202628cfe
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
libyul/optimiser/ExpressionJoiner.cpp
View File

@ -56,7 +56,7 @@ void ExpressionJoiner::operator()(Switch& _switch)
{ {
visit(*_switch.expression); visit(*_switch.expression);
for (auto& _case: _switch.cases) for (auto& _case: _switch.cases)
// Do not visit the case expression, nothing to unbreak there. // Do not visit the case expression, nothing to join there.
(*this)(_case.body); (*this)(_case.body);
} }

4
libyul/optimiser/ExpressionSplitter.cpp
View File

@ -56,14 +56,14 @@ void ExpressionSplitter::operator()(Switch& _switch)
{ {
outlineExpression(*_switch.expression); outlineExpression(*_switch.expression);
for (auto& _case: _switch.cases) for (auto& _case: _switch.cases)
// Do not visit the case expression, nothing to break there. // Do not visit the case expression, nothing to split there.
(*this)(_case.body); (*this)(_case.body);
} }
void ExpressionSplitter::operator()(ForLoop& _loop) void ExpressionSplitter::operator()(ForLoop& _loop)
{ {
(*this)(_loop.pre); (*this)(_loop.pre);
// Do not visit the condition because we cannot break expressions there. // Do not visit the condition because we cannot split expressions there.
(*this)(_loop.post); (*this)(_loop.post);
(*this)(_loop.body); (*this)(_loop.body);
} }

2
test/libyul/yulOptimizerTests/fullInliner/inside_condition.yul
View File

@ -1,4 +1,4 @@
// This tests that breaking the expression inside the condition works properly. // This tests that splitting the expression inside the condition works properly.
{ {
if gt(f(mload(1)), mload(0)) { if gt(f(mload(1)), mload(0)) {
sstore(0, 2) sstore(0, 2)