aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMathias Baumann <marenz@supradigital.org>2018-12-11 02:03:47 +0800
committerMathias Baumann <marenz@supradigital.org>2018-12-11 02:03:47 +0800
commitcb935fe9086b60295f97c5b8a5fd0ea8245d145e (patch)
tree60d9ee91441932526bc86384990cb9c99a0f3039
parent2f6dc2e773148f63f4e2b9d9b3f9bb7eb092fde8 (diff)
downloaddexon-solidity-cb935fe9086b60295f97c5b8a5fd0ea8245d145e.tar
dexon-solidity-cb935fe9086b60295f97c5b8a5fd0ea8245d145e.tar.gz
dexon-solidity-cb935fe9086b60295f97c5b8a5fd0ea8245d145e.tar.bz2
dexon-solidity-cb935fe9086b60295f97c5b8a5fd0ea8245d145e.tar.lz
dexon-solidity-cb935fe9086b60295f97c5b8a5fd0ea8245d145e.tar.xz
dexon-solidity-cb935fe9086b60295f97c5b8a5fd0ea8245d145e.tar.zst
dexon-solidity-cb935fe9086b60295f97c5b8a5fd0ea8245d145e.zip
Use curly-brackets initialization
-rw-r--r--libevmasm/Assembly.cpp14
-rw-r--r--libevmasm/BlockDeduplicator.cpp14
2 files changed, 14 insertions, 14 deletions
diff --git a/libevmasm/Assembly.cpp b/libevmasm/Assembly.cpp
index 2654eae2..231eed93 100644
--- a/libevmasm/Assembly.cpp
+++ b/libevmasm/Assembly.cpp
@@ -353,14 +353,14 @@ AssemblyItem Assembly::namedTag(string const& _name)
assertThrow(!_name.empty(), AssemblyException, "Empty named tag.");
if (!m_namedTags.count(_name))
m_namedTags[_name] = size_t(newTag().data());
- return AssemblyItem(Tag, m_namedTags.at(_name));
+ return AssemblyItem{Tag, m_namedTags.at(_name)};
}
AssemblyItem Assembly::newPushLibraryAddress(string const& _identifier)
{
h256 h(dev::keccak256(_identifier));
m_libraries[h] = _identifier;
- return AssemblyItem(PushLibraryAddress, h);
+ return AssemblyItem{PushLibraryAddress, h};
}
Assembly& Assembly::optimise(bool _enable, EVMVersion _evmVersion, bool _isCreation, size_t _runs)
@@ -415,14 +415,14 @@ map<u256, u256> Assembly::optimiseInternal(
if (_settings.runJumpdestRemover)
{
- JumpdestRemover jumpdestOpt(m_items);
+ JumpdestRemover jumpdestOpt{m_items};
if (jumpdestOpt.optimise(_tagsReferencedFromOutside))
count++;
}
if (_settings.runPeephole)
{
- PeepholeOptimiser peepOpt(m_items);
+ PeepholeOptimiser peepOpt{m_items};
while (peepOpt.optimise())
{
count++;
@@ -433,7 +433,7 @@ map<u256, u256> Assembly::optimiseInternal(
// This only modifies PushTags, we have to run again to actually remove code.
if (_settings.runDeduplicate)
{
- BlockDeduplicator dedup(m_items);
+ BlockDeduplicator dedup{m_items};
if (dedup.deduplicate())
{
tagReplacements.insert(dedup.replacedTags().begin(), dedup.replacedTags().end());
@@ -448,13 +448,13 @@ map<u256, u256> Assembly::optimiseInternal(
// function types that can be stored in storage.
AssemblyItems optimisedItems;
- bool usesMSize = (find(m_items.begin(), m_items.end(), AssemblyItem(Instruction::MSIZE)) != m_items.end());
+ bool usesMSize = (find(m_items.begin(), m_items.end(), AssemblyItem{Instruction::MSIZE}) != m_items.end());
auto iter = m_items.begin();
while (iter != m_items.end())
{
KnownState emptyState;
- CommonSubexpressionEliminator eliminator(emptyState);
+ CommonSubexpressionEliminator eliminator{emptyState};
auto orig = iter;
iter = eliminator.feedItems(iter, m_items.end(), usesMSize);
bool shouldReplace = false;
diff --git a/libevmasm/BlockDeduplicator.cpp b/libevmasm/BlockDeduplicator.cpp
index b7c69531..ca439925 100644
--- a/libevmasm/BlockDeduplicator.cpp
+++ b/libevmasm/BlockDeduplicator.cpp
@@ -41,7 +41,7 @@ bool BlockDeduplicator::deduplicate()
// Virtual tag that signifies "the current block" and which is used to optimise loops.
// We abort if this virtual tag actually exists.
- AssemblyItem pushSelf(PushTag, u256(-4));
+ AssemblyItem pushSelf{PushTag, u256(-4)};
if (
std::count(m_items.cbegin(), m_items.cend(), pushSelf.tag()) ||
std::count(m_items.cbegin(), m_items.cend(), pushSelf.pushTag())
@@ -55,17 +55,17 @@ bool BlockDeduplicator::deduplicate()
// To compare recursive loops, we have to already unify PushTag opcodes of the
// block's own tag.
- AssemblyItem pushFirstTag(pushSelf);
- AssemblyItem pushSecondTag(pushSelf);
+ AssemblyItem pushFirstTag{pushSelf};
+ AssemblyItem pushSecondTag{pushSelf};
if (_i < m_items.size() && m_items.at(_i).type() == Tag)
pushFirstTag = m_items.at(_i).pushTag();
if (_j < m_items.size() && m_items.at(_j).type() == Tag)
pushSecondTag = m_items.at(_j).pushTag();
- BlockIterator first(m_items.begin() + _i, m_items.end(), &pushFirstTag, &pushSelf);
- BlockIterator second(m_items.begin() + _j, m_items.end(), &pushSecondTag, &pushSelf);
- BlockIterator end(m_items.end(), m_items.end());
+ BlockIterator first{m_items.begin() + _i, m_items.end(), &pushFirstTag, &pushSelf};
+ BlockIterator second{m_items.begin() + _j, m_items.end(), &pushSecondTag, &pushSelf};
+ BlockIterator end{m_items.end(), m_items.end()};
if (first != end && (*first).type() == Tag)
++first;
@@ -126,7 +126,7 @@ BlockDeduplicator::BlockIterator& BlockDeduplicator::BlockIterator::operator++()
{
if (it == end)
return *this;
- if (SemanticInformation::altersControlFlow(*it) && *it != AssemblyItem(Instruction::JUMPI))
+ if (SemanticInformation::altersControlFlow(*it) && *it != AssemblyItem{Instruction::JUMPI})
it = end;
else
{