-
Notifications
You must be signed in to change notification settings - Fork 21
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
Tracking issue for phase 4 advancement #43
Comments
A while ago I heard about the intent to include 64-bit tables as well, in order to be able to compile function pointers consistently. What became of that? |
That request came from @matthias-blume who is working on an experimental wasm to native compiler without a sandbox. We have a workaround in llvm that truncate all function pointers before call indirect. It add an instruction for every call indirect in the program so it could save a bit on code size and complexity if we could remove it. Do you think it makes sense to roll it into this proposal? |
Yeah, from my perspective, Wasm64 is incomplete without it, and it leaves the language in an odd space. Do you think it would still be realistic to extend the proposal? |
What's the status of the spec document? It would be good to get the changes in well ahead of the phase 4 vote at the f2f meeting. |
I need to pick up this open PR: #50 |
Now with only the table64 extension remained, can we get an estimate please? |
We are just waiting on the second implementation of table64 now (in spidermonkey). My understanding is that is underway, so we should be able to vote on phase in next one or two meetings. |
@sbc100 Should the following two issues be added to this checklist?
|
@bvisness, I'd think so. |
@sbc100 More issues / PRs that I think should be on this list:
There are also the other scattered spec PRs I've been working on, but those are basically spec bugfixes and I don't think we need to track them here. |
#80 should be on this list as well, presumably. |
Indeed. Added #80, hopefully we can get those updates done by next meeting and vote on this. |
We are now phase 4! |
Index Type
to something else, perhapsOffset Type
#67The text was updated successfully, but these errors were encountered: