Stream: general

Topic: wasmtime issue triaging


view this post on Zulip bjorn3 (Feb 03 2021 at 17:27):

I have been left triage comments on a dozen issues which I think should be closed. Can someone take a look? And is it ok if I continue triaging the rest of the issues that I think should be closed?
Edit: seems like @Chris Fallin is already taking a look.

view this post on Zulip bjorn3 (Feb 03 2021 at 17:40):

And is it ok if I continue triaging the rest of the issues that I think should be closed?

@Chris Fallin?

view this post on Zulip bjorn3 (Feb 03 2021 at 17:53):

I got another 22 issues that can be closed in addition to the few that haven't yet been closed. Should I comment on them or list them here?

view this post on Zulip Chris Fallin (Feb 03 2021 at 18:18):

@bjorn3 please feel free to keep commenting, indeed

view this post on Zulip Chris Fallin (Feb 03 2021 at 18:18):

or you can close directly if it's clearly addressed -- we can always reopen if we disagree :-)

view this post on Zulip bjorn3 (Feb 03 2021 at 18:28):

I don't have permission to close any issues that are not my own.

view this post on Zulip bjorn3 (Feb 03 2021 at 18:29):

I am done with triage of the most obvious issues.

view this post on Zulip Chris Fallin (Feb 03 2021 at 18:29):

Ah, sorry, didn't realize; in that case I'll keep following up (thanks again for this cleanup effort!)

view this post on Zulip bjorn3 (Feb 03 2021 at 18:30):

no problem

view this post on Zulip bjorn3 (Feb 03 2021 at 18:40):

I noticed that there are 114 issues without label indicating if it is about cranelift, wasmtime, lightbeam, peepmatic, ...: https://github.com/bytecodealliance/wasmtime/issues?page=1&q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+-label%3Acranelift+-label%3Awasi%3Aapi+-label%3Alightbeam+-label%3Awasi%3Aimpl+-label%3Acranelift%3Aarea%3Amachinst+-label%3Awasi%3Atests+-label%3Awasmtime%3Ac-api+-label%3Awasmtime+-label%3Awasmtime%3Aapi+-label%3Afuzzing+-label%3Acranelift%3Adiscussion+-label%3Awasm-proposal%3Areference-types+-label%3Acranelift%3Aarea%3Apeepmatic+-label%3Awasmtime%3Adebugging

Standalone JIT-style runtime for WebAssembly, using Cranelift - bytecodealliance/wasmtime

view this post on Zulip Andrew Brown (Feb 03 2021 at 19:44):

glancing at that, I think we might want to expand our label set: benchmarking, ci, optimization...

view this post on Zulip Andrew Brown (Feb 03 2021 at 19:45):

but, yeah, we should start labeling some of those

view this post on Zulip bjorn3 (Feb 03 2021 at 20:19):

What is the status of wasm2obj? There are several issues about it open, but I don't know if it is still supported.

view this post on Zulip bjorn3 (Feb 03 2021 at 20:21):

Also the browser wasi polyfill is pretty much unsupported and doesn't support the latest wasi, right?

view this post on Zulip Andrew Brown (Feb 03 2021 at 20:35):

@Dan Gohman?

view this post on Zulip bjorn3 (Feb 03 2021 at 21:04):

48 issues closed. 333 left.

view this post on Zulip bjorn3 (Feb 03 2021 at 21:04):

/me calls it a day

view this post on Zulip Dan Gohman (Feb 03 2021 at 21:16):

I don't actually know the current status of wasm2obj. @Yury Delendik; do you know?

view this post on Zulip Yury Delendik (Feb 03 2021 at 21:18):

I did not look into that for 3-4 months, no. It produced valid obj files with DWARF info when needed

view this post on Zulip Yury Delendik (Feb 03 2021 at 21:37):

/me used wasm2obj only to troubleshot DWARF issues

view this post on Zulip Dan Gohman (Feb 04 2021 at 18:49):

Also, yes, the wasi browser polyfill in the wasmtime tree is unsupported right now

view this post on Zulip Dan Gohman (Feb 04 2021 at 18:50):

In theory we could build a new polyfill in a similar way, however one of the things we learned since building that was that hand-writtenJS polyfills that various people have end up being much simpler and smaller

view this post on Zulip bjorn3 (Mar 20 2021 at 18:18):

Did another pass and opened a few PR's for easy to fix issues.

view this post on Zulip bjorn3 (Jun 22 2024 at 19:35):

Another pass. The following can be closed I think:

Currently, binemit has its own CodeSink trait for writing binary data. It isn't entirely satisfying, in part because it's an unsafe interface -- it doesn't perform bounds checking on the underlying...
I'd like to use CraneLift as a JIT backend for a standalone WebAssembly embedding. However, from browsing the documentation I'm not sure how to do this, or even if it's fully possible yet. An examp...
This is the continuation of the closed PR bytecodealliance/cranelift#624. Spidermonkey uses a pinned register to contain the HeapBase on x64. It is supposed to be callee preserved, so we need a way...
I'm confused, if calling __wasi_path_open, should you always pass this flag? Where do you get the initial permission to open a file?
I am interested in building Wasmtime against a Cranelift with all architectures disabled except for the target architecture. From my research, it seems that the way to accomplish this would be to r...
To quote the spec: __WASI_RIGHT_PATH_FILESTAT_SET_SIZE The right to change a file's size (there is no __wasi_path_filestat_set_size()). If __WASI_RIGHT_PATH_OPEN is set, includes the right to invok...
The link to the Javascripte polyfill source code in https://github.com/bytecodealliance/wasmtime/blob/master/docs/WASI-intro.md#the-browser-polyfill seems to be broken. I got to the document via wa...

view this post on Zulip bjorn3 (Jun 22 2024 at 19:37):

If someone with appropriate permissions could take a look, that did be appreciated.

view this post on Zulip Chris Fallin (Jun 22 2024 at 23:15):

Awesome, thanks for making the pass! I closed four of them; the two WASI ones (143, 273) I'll delegate to someone else (@Alex Crichton ? @Dan Gohman ?) for final evaluation


Last updated: Nov 22 2024 at 16:03 UTC