Skip to content
This repository was archived by the owner on Apr 25, 2025. It is now read-only.

Status and path to Phase 4 #18

Open
yuri91 opened this issue Jun 14, 2023 · 10 comments
Open

Status and path to Phase 4 #18

yuri91 opened this issue Jun 14, 2023 · 10 comments

Comments

@yuri91
Copy link
Contributor

yuri91 commented Jun 14, 2023

I would like to see this proposal going forward, since it's a prerequisite for the branch hinting proposal.

We recently solved the main issue of missing tests (#15) by adding to the interpreter a way to do (optional) extended checks on custom sections/annotations (#17).

There are still 3 open issues, but I am not sure if they are active concerns or not.

Is there anything that still needs to be done to go forward? If so, I am happy to help.

@rossberg
Copy link
Member

I think the only real open question is how this would interact with future repeated sections (#11). But I think we at least have a possible plan, as stated on the issue.

Other than that, this proposal was mainly held back by various meta considerations (#5), most of which were actually outside the scope of this proposal. I'd be happy to move forward.

@yuri91
Copy link
Contributor Author

yuri91 commented Jun 15, 2023

Thanks for the recap!
Nice to know that there are no real blockers.

@rossberg
Copy link
Member

Well, not from my perspective anyway. If it was for me, this could have been merged years ago. ;)

@tlively
Copy link
Member

tlively commented Apr 3, 2024

The new text parser in Binaryen (not quite enabled by default) supports parsing annotations in general, although not the name annotations or the syntax expressing arbitrary custom sections. Assuming we will remove the name annotations as discussed in #21, then we should be able to declare the phase 4 toolchain requirement fulfilled once Binaryen enables the new parser by default and adds support for the arbitrary custom section syntax.

Are we planning to waive the Web engine implementation requirement?

@tlively
Copy link
Member

tlively commented Apr 23, 2024

Bump. @rossberg, I'd like to get this voted to phase 4 at the June CG meeting if possible. What else needs to happen before that? Do we need to have a CG discussion to resolve the question about the @name annotations?

@rossberg
Copy link
Member

rossberg commented Apr 24, 2024

Sounds good! Yes, the only thing we need to resolve is whether to keep @name as the textual representation for the name section. We can try to resolve that beforehand or at the June meeting.

@tlively
Copy link
Member

tlively commented Apr 24, 2024

Great, I've proposed an agenda item for the next CG meeting so that I can find out sooner rather than later if I'm going to need to implement name annotations in Binaryen before the June meeting. WebAssembly/meetings#1554

@rossberg
Copy link
Member

I'm afraid I won't be able to attend the next meeting, since I will be at a meeting in Japan.

@tlively
Copy link
Member

tlively commented Apr 24, 2024

How about the next one on May 21?

@rossberg
Copy link
Member

That works.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants