Thank you very much! On Thu, Jun 18, 2020 at 9:22 PM <slightly...@chromium.org> wrote:
> LGTM3 > > On Thursday, June 18, 2020 at 12:17:52 PM UTC-7, Mike West wrote: >> >> LGTM2. >> >> On Tuesday, June 16, 2020 at 5:54:30 PM UTC+2 Andreas Haas wrote: >> >>> Launch bug: >>> https://bugs.chromium.org/p/chromium/issues/detail?id=1094837 >>> >>> On Friday, June 12, 2020 at 8:42:11 AM UTC+2 Andreas Haas wrote: >>> >>>> Spec >>>> >>>> https://github.com/WebAssembly/JS-BigInt-integration >>>> Contact emails >>>> ah...@google.com, ecmzi...@google.com >>>> >>>> Summary >>>> >>>> The "JavaScript BigInt to WebAssembly i64 integration >>>> <https://github.com/WebAssembly/JS-BigInt-integration>" proposal >>>> extends the JavaScript-WebAssembly API to allow passing I64 values from >>>> WebAssembly to JavaScript, and vice versa. The representation of I64 values >>>> in JavaScript is BigInt. >>>> >>>> Without this proposal, an exception is thrown every time an I64 value >>>> is passed over the WebAssembly - JavaScript API. >>>> >>>> Is this feature supported on all six Blink platforms (Windows, Mac, >>>> Linux, Chrome OS, Android, and Android WebView)? >>>> >>>> Yes. >>>> >>>> Interoperability and Compatibility risk >>>> This is a finished WebAssembly proposal >>>> <https://github.com/WebAssembly/proposals/blob/master/finished-proposals.md> >>>> and has already been integrated into the core WebAssembly spec. The >>>> proposal is implemented in Chrome and Firefox and passes the spec tests in >>>> both. Changing the proposal would require changing the WebAssembly spec. >>>> >>>> Firefox: Implemented >>>> <https://bugzilla.mozilla.org/show_bug.cgi?id=1608772> >>>> Safari: No signals >>>> >>>> Is this feature fully tested? >>>> >>>> Yes, tested in wasm mjsunit tests: >>>> >>>> https://cs.chromium.org/chromium/src/v8/test/mjsunit/wasm/bigint.js >>>> >>>> >>>> Also passes the wasm spec tests: >>>> https://github.com/WebAssembly/JS-BigInt-integration/tree/master/test/js-api >>>> >>>> >>>> The feature has also been staged for several months now to get >>>> ClusterFuzz coverage. >>>> >>>> >>>> Tracking bug >>>> >>>> https://crbug.com/v8/7741 <https://crbug.com/v8/7747> >>>> >>>> Entry on the feature dashboard <http://www.chromestatus.com/> >>>> >>>> https://chromestatus.com/feature/5648655109324800 >>>> >>>> Requesting approval to ship? >>>> >>>> Yes. >>>> >>> -- -- v8-users mailing list v8-users@googlegroups.com http://groups.google.com/group/v8-users --- You received this message because you are subscribed to the Google Groups "v8-users" group. To unsubscribe from this group and stop receiving emails from it, send an email to v8-users+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/v8-users/CAELSTvdh7ziALVnUvgPhJcsTz5%3Dmcx9vMi-X8tgrs5ALO%3Dc3yA%40mail.gmail.com.
