Skip to content

[MetaModel] Switch FnParam to new Typing System - #118

Open
jokap11 wants to merge 1 commit into
coredsl2from
bugfix/FnParamPointer
Open

[MetaModel] Switch FnParam to new Typing System#118
jokap11 wants to merge 1 commit into
coredsl2from
bugfix/FnParamPointer

Conversation

@jokap11

@jokap11 jokap11 commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Fixes function signatures with pointers ...
Should support this with the new typing system:

functions {
        extern unsigned<8> etiss_vload_encoded_unitstride(char* pV, unsigned<16> pVTYPE, unsigned<8> pVm, unsigned<16> pEEW, unsigned<8> pVd, unsigned<16> pVSTART, unsigned<16> pVLEN, unsigned<16> pVL, unsigned<64> pMSTART) [[etiss_needs_arch]];
        ...
    }

Fixes function signatures with pointers ...
@PhilippvK

Copy link
Copy Markdown
Member

Storing the actual type info in the FnParam class instead of just the size & kind is definitely the way to go. This was one of the main motivations to du the refactoring in the first place :D

thank you for the patch @jokap11, I will test it tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants