Closed Bug 1770688 Opened 3 years ago Closed 3 years ago

Move some common testing functions to jit-test/lib/wasm.js

Categories

(Core :: JavaScript: WebAssembly, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
102 Branch
Tracking Status
firefox102 --- fixed

People

(Reporter: lth, Assigned: lth)

References

Details

Attachments

(1 file)

A number of testing helpers - most notably iota - are duplicated in numerous wasm test files. Move these helpers to wasm.lib.

Move a number of Array-oriented testing functions to lib/wasm.js, as we
keep copying them from file to file, with variations.

The main sticking point here was actually that the simd tests include
a common testing file using an include: directive, but this directive
is processed before the common include: directive that loads wasm.js.

Unfortunately the present patch creates a dependency wherein the simd
common testing file will depend on wasm.js being loaded first.

To fix the dependency order the simd tests were updated to load the
common testing file explicitly, and this load will happen after
wasm.js is loaded.

Pushed by lhansen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/280a33091659 Move common testing functions to lib/wasm.js. r=jseward
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: