cerc-io/solidity
15
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

Fix "slot" access via mapping reference in assembly

Browse Source
This commit is contained in:
wechman 2022-08-16 12:34:17 +02:00
parent bb41ddd7ab
commit cf3bae0839
4 changed files with 40 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
libsolidity/ast/Types.cpp
View File

@ -3789,6 +3789,11 @@ TypeResult MappingType::interfaceType(bool _inLibrary) const
return this; return this;
} }
std::vector<std::tuple<std::string, Type const*>> MappingType::makeStackItems() const
{
return {std::make_tuple("slot", TypeProvider::uint256())};
}
string TypeType::richIdentifier() const string TypeType::richIdentifier() const
{ {
return "t_type" + identifierList(actualType()); return "t_type" + identifierList(actualType());

2
libsolidity/ast/Types.h
View File

@ -1528,6 +1528,8 @@ public:
bool hasSimpleZeroValueInMemory() const override { solAssert(false, ""); } bool hasSimpleZeroValueInMemory() const override { solAssert(false, ""); }
bool nameable() const override { return true; } bool nameable() const override { return true; }
std::vector<std::tuple<std::string, Type const*>> makeStackItems() const override;
Type const* keyType() const { return m_keyType; } Type const* keyType() const { return m_keyType; }
Type const* valueType() const { return m_valueType; } Type const* valueType() const { return m_valueType; }

3
libsolidity/codegen/ir/IRGeneratorForStatements.cpp
View File

@ -173,10 +173,9 @@ private:
{ {
solAssert(suffix == "slot" || suffix == "offset"); solAssert(suffix == "slot" || suffix == "offset");
solAssert(varDecl->isLocalVariable()); solAssert(varDecl->isLocalVariable());
solAssert(!varDecl->type()->isValueType());
if (suffix == "slot") if (suffix == "slot")
value = IRVariable{*varDecl}.part("slot").name(); value = IRVariable{*varDecl}.part("slot").name();
else if (varDecl->type()->isValueType())
value = IRVariable{*varDecl}.part("offset").name();
else else
{ {
solAssert(!IRVariable{*varDecl}.hasPart("offset")); solAssert(!IRVariable{*varDecl}.hasPart("offset"));

32
test/libsolidity/semanticTests/inlineAssembly/slot_access_via_mapping_pointer.sol Normal file
View File

@ -0,0 +1,32 @@
contract C {
mapping(uint => uint) private m0;
mapping(uint => uint) private m1;
mapping(uint => uint) private m2;
function f(uint i) public returns (uint slot, uint offset) {
mapping(uint => uint) storage m0Ptr = m0;
mapping(uint => uint) storage m1Ptr = m1;
mapping(uint => uint) storage m2Ptr = m2;
assembly {
switch i
case 1 {
slot := m1Ptr.slot
offset := m1Ptr.offset
}
case 2 {
slot := m2Ptr.slot
offset := m2Ptr.offset
}
default {
slot := m0Ptr.slot
offset := m0Ptr.offset
}
}
}
}
// ----
// f(uint256): 0 -> 0, 0
// f(uint256): 1 -> 1, 0
// f(uint256): 2 -> 2, 0