Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error in WASM_INIT_FUNCS docs #137

Open
theidiotmachine opened this issue Feb 9, 2020 · 1 comment
Open

Error in WASM_INIT_FUNCS docs #137

theidiotmachine opened this issue Feb 9, 2020 · 1 comment

Comments

@theidiotmachine
Copy link

Hey. The WASM_INIT_FUNCS linker subsection is documented as being a count and then a 'sequence of symbol indices'. However, the latest implementation of wasm-lld reads pairs of varuint32s; the first as a priority, the second the symbol index, here.

I think this came from this which describes such a schema.

It means that if you follow this spec, the linker will reject you, so I take it this is an error in this doc?

@sbc100
Copy link
Member

sbc100 commented Feb 11, 2020

Yup, looks like that docs need an update.

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

No branches or pull requests

2 participants