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

Add helper to see if a switch has a default case.

Browse Source
This commit is contained in:
chriseth 2022-03-15 14:24:42 +01:00
parent c6ac1625bd
commit 28593839d9
3 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
libsolidity/analysis/ControlFlowBuilder.cpp
View File

@ -457,8 +457,7 @@ void ControlFlowBuilder::operator()(yul::Switch const& _switch)
} }
mergeFlow(nodes); mergeFlow(nodes);
bool hasDefault = util::contains_if(_switch.cases, [](yul::Case const& _case) { return !_case.value; }); if (!hasDefaultCase(_switch))
if (!hasDefault)
connect(beforeSwitch, m_currentNode); connect(beforeSwitch, m_currentNode);
} }

9
libyul/AST.h
View File

@ -144,4 +144,13 @@ template <class... Args> inline std::shared_ptr<DebugData const> debugDataOf(std
return std::visit([](auto const& _arg) { return debugDataOf(_arg); }, _node); return std::visit([](auto const& _arg) { return debugDataOf(_arg); }, _node);
} }
inline bool hasDefaultCase(Switch const& _switch)
{
return std::any_of(
_switch.cases.begin(),
_switch.cases.end(),
[](Case const& _case) { return !_case.value; }
);
}
} }

8
libyul/optimiser/ControlFlowSimplifier.cpp
View File

@ -60,13 +60,7 @@ void removeEmptyDefaultFromSwitch(Switch& _switchStmt)
void removeEmptyCasesFromSwitch(Switch& _switchStmt) void removeEmptyCasesFromSwitch(Switch& _switchStmt)
{ {
bool hasDefault = std::any_of( if (hasDefaultCase(_switchStmt))
_switchStmt.cases.begin(),
_switchStmt.cases.end(),
[](Case const& _case) { return !_case.value; }
);
if (hasDefault)
return; return;
ranges::actions::remove_if( ranges::actions::remove_if(