mirror of
https://github.com/ethereum/solidity
synced 2023-10-03 13:03:40 +00:00
Use shift helper
This commit is contained in:
parent
0494fa98c0
commit
c7ae042114
@ -371,10 +371,10 @@ void CompilerUtils::combineExternalFunctionType(bool _leftAligned)
|
|||||||
m_context << u256(0xffffffffUL) << Instruction::AND << Instruction::SWAP1;
|
m_context << u256(0xffffffffUL) << Instruction::AND << Instruction::SWAP1;
|
||||||
if (!_leftAligned)
|
if (!_leftAligned)
|
||||||
m_context << ((u256(1) << 160) - 1) << Instruction::AND;
|
m_context << ((u256(1) << 160) - 1) << Instruction::AND;
|
||||||
m_context << (u256(1) << 32) << Instruction::MUL;
|
leftShiftNumberOnStack(u256(1) << 32);
|
||||||
m_context << Instruction::OR;
|
m_context << Instruction::OR;
|
||||||
if (_leftAligned)
|
if (_leftAligned)
|
||||||
m_context << (u256(1) << 64) << Instruction::MUL;
|
leftShiftNumberOnStack(u256(1) << 64);
|
||||||
}
|
}
|
||||||
|
|
||||||
void CompilerUtils::pushCombinedFunctionEntryLabel(Declaration const& _function)
|
void CompilerUtils::pushCombinedFunctionEntryLabel(Declaration const& _function)
|
||||||
@ -425,7 +425,7 @@ void CompilerUtils::convertType(
|
|||||||
// conversion from bytes to integer. no need to clean the high bit
|
// conversion from bytes to integer. no need to clean the high bit
|
||||||
// only to shift right because of opposite alignment
|
// only to shift right because of opposite alignment
|
||||||
IntegerType const& targetIntegerType = dynamic_cast<IntegerType const&>(_targetType);
|
IntegerType const& targetIntegerType = dynamic_cast<IntegerType const&>(_targetType);
|
||||||
m_context << (u256(1) << (256 - typeOnStack.numBytes() * 8)) << Instruction::SWAP1 << Instruction::DIV;
|
rightShiftNumberOnStack(u256(1) << (256 - typeOnStack.numBytes() * 8), false);
|
||||||
if (targetIntegerType.numBits() < typeOnStack.numBytes() * 8)
|
if (targetIntegerType.numBits() < typeOnStack.numBytes() * 8)
|
||||||
convertType(IntegerType(typeOnStack.numBytes() * 8), _targetType, _cleanupNeeded);
|
convertType(IntegerType(typeOnStack.numBytes() * 8), _targetType, _cleanupNeeded);
|
||||||
}
|
}
|
||||||
@ -476,7 +476,7 @@ void CompilerUtils::convertType(
|
|||||||
if (auto typeOnStack = dynamic_cast<IntegerType const*>(&_typeOnStack))
|
if (auto typeOnStack = dynamic_cast<IntegerType const*>(&_typeOnStack))
|
||||||
if (targetBytesType.numBytes() * 8 > typeOnStack->numBits())
|
if (targetBytesType.numBytes() * 8 > typeOnStack->numBits())
|
||||||
cleanHigherOrderBits(*typeOnStack);
|
cleanHigherOrderBits(*typeOnStack);
|
||||||
m_context << (u256(1) << (256 - targetBytesType.numBytes() * 8)) << Instruction::MUL;
|
leftShiftNumberOnStack(u256(1) << (256 - targetBytesType.numBytes() * 8));
|
||||||
}
|
}
|
||||||
else if (targetTypeCategory == Type::Category::Enum)
|
else if (targetTypeCategory == Type::Category::Enum)
|
||||||
{
|
{
|
||||||
@ -987,9 +987,9 @@ unsigned CompilerUtils::loadFromMemoryHelper(Type const& _type, bool _fromCallda
|
|||||||
bool leftAligned = _type.category() == Type::Category::FixedBytes;
|
bool leftAligned = _type.category() == Type::Category::FixedBytes;
|
||||||
// add leading or trailing zeros by dividing/multiplying depending on alignment
|
// add leading or trailing zeros by dividing/multiplying depending on alignment
|
||||||
u256 shiftFactor = u256(1) << ((32 - numBytes) * 8);
|
u256 shiftFactor = u256(1) << ((32 - numBytes) * 8);
|
||||||
m_context << shiftFactor << Instruction::SWAP1 << Instruction::DIV;
|
rightShiftNumberOnStack(shiftFactor, false);
|
||||||
if (leftAligned)
|
if (leftAligned)
|
||||||
m_context << shiftFactor << Instruction::MUL;
|
leftShiftNumberOnStack(shiftFactor);
|
||||||
}
|
}
|
||||||
if (_fromCalldata)
|
if (_fromCalldata)
|
||||||
convertType(_type, _type, true, false, true);
|
convertType(_type, _type, true, false, true);
|
||||||
|
@ -526,7 +526,7 @@ bool ExpressionCompiler::visit(FunctionCall const& _functionCall)
|
|||||||
|
|
||||||
if (m_context.runtimeContext())
|
if (m_context.runtimeContext())
|
||||||
// We have a runtime context, so we need the creation part.
|
// We have a runtime context, so we need the creation part.
|
||||||
m_context << (u256(1) << 32) << Instruction::SWAP1 << Instruction::DIV;
|
utils().rightShiftNumberOnStack(u256(1) << 32, false);
|
||||||
else
|
else
|
||||||
// Extract the runtime part.
|
// Extract the runtime part.
|
||||||
m_context << ((u256(1) << 32) - 1) << Instruction::AND;
|
m_context << ((u256(1) << 32) - 1) << Instruction::AND;
|
||||||
@ -1269,7 +1269,7 @@ bool ExpressionCompiler::visit(IndexAccess const& _indexAccess)
|
|||||||
m_context.appendConditionalInvalid();
|
m_context.appendConditionalInvalid();
|
||||||
|
|
||||||
m_context << Instruction::BYTE;
|
m_context << Instruction::BYTE;
|
||||||
m_context << (u256(1) << (256 - 8)) << Instruction::MUL;
|
utils().leftShiftNumberOnStack(u256(1) << (256 - 8));
|
||||||
}
|
}
|
||||||
else if (baseType.category() == Type::Category::TypeType)
|
else if (baseType.category() == Type::Category::TypeType)
|
||||||
{
|
{
|
||||||
|
Loading…
Reference in New Issue
Block a user