zink icon indicating copy to clipboard operation
zink copied to clipboard

Support struct as function parameters

Open clearloop opened this issue 1 year ago • 3 comments
trafficstars

Describe the feature

similar to the support of dynamic arguments

Solution

need to calculate the length of struct in compilation from function signatures

clearloop avatar Oct 31 '24 00:10 clearloop

@clearloop currently working on this, I don't think I understand this fully lol

malik672 avatar Nov 27 '24 06:11 malik672

@clearloop currently working on this, I don't think I understand this fully lol

Thank you for your contribution!

This issue requires some pre-requirements, for example the support of #253 which is pretty complex in the memory model, if you'd like to dive into the memory model, I'd suggest to try #249 if you understand the dynamic memory in solidity correctly! you can imagine that excepting the basic opcode mappings, zink is like a HUFF library ^ ^

btw, I hope you can focus on the issues assigned to you previously first )) once you complete #255 and #256, it will be easy for you to handle all of the derived macro related stuffs in zink, it's a huge part indeed ))

feel free to chat about anything related in our telegram group!

clearloop avatar Nov 27 '24 06:11 clearloop

@clearloop currently working on this, I don't think I understand this fully lol

Thank you for your contribution!

This issue requires some pre-requirements, for example the support of #253 which is pretty complex in the memory model, if you'd like to dive into the memory model, I'd suggest to try #249 if you understand the dynamic memory in solidity correctly! you can imagine that excepting the basic opcode mappings, zink is like a HUFF library ^ ^

btw, I hope you can focus on the issues assigned to you previously first )) once you complete #255 and #256, it will be easy for you to handle all of the derived macro related stuffs in zink, it's a huge part indeed ))

feel free to chat about anything related in our telegram group!

will leave it then

malik672 avatar Nov 27 '24 06:11 malik672