From 0664772ece60749e606ee94eb073206340cbb08c Mon Sep 17 00:00:00 2001 From: chriseth Date: Mon, 15 Oct 2018 12:51:50 +0200 Subject: [PATCH] Some renaming leftovers. --- libyul/optimiser/ExpressionJoiner.cpp | 2 +- libyul/optimiser/ExpressionSplitter.cpp | 4 ++-- .../libyul/yulOptimizerTests/fullInliner/inside_condition.yul | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libyul/optimiser/ExpressionJoiner.cpp b/libyul/optimiser/ExpressionJoiner.cpp index 3dfab3716..c39574979 100644 --- a/libyul/optimiser/ExpressionJoiner.cpp +++ b/libyul/optimiser/ExpressionJoiner.cpp @@ -56,7 +56,7 @@ void ExpressionJoiner::operator()(Switch& _switch) { visit(*_switch.expression); 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); } diff --git a/libyul/optimiser/ExpressionSplitter.cpp b/libyul/optimiser/ExpressionSplitter.cpp index dab0545f0..a2ecc546d 100644 --- a/libyul/optimiser/ExpressionSplitter.cpp +++ b/libyul/optimiser/ExpressionSplitter.cpp @@ -56,14 +56,14 @@ void ExpressionSplitter::operator()(Switch& _switch) { outlineExpression(*_switch.expression); 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); } void ExpressionSplitter::operator()(ForLoop& _loop) { (*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.body); } diff --git a/test/libyul/yulOptimizerTests/fullInliner/inside_condition.yul b/test/libyul/yulOptimizerTests/fullInliner/inside_condition.yul index ae25e9a3e..76b6054b3 100644 --- a/test/libyul/yulOptimizerTests/fullInliner/inside_condition.yul +++ b/test/libyul/yulOptimizerTests/fullInliner/inside_condition.yul @@ -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)) { sstore(0, 2)