Closed Bug 1292126 Opened 5 years ago Closed 5 years ago

wasm: Enable more test cases

Categories

(Core :: JavaScript Engine: JIT, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: bbouvier, Assigned: bbouvier)

References

Details

Attachments

(1 file)

Apparently some test cases were written before features were implemented, and those were remaining dormant. There are a few of them we can pass now, so let's just enable them.

Double-check me, but the ml-proto and our implementation seem okay that a function return something when no (result) is set, at the moment.
Blocks: wasm
Attached patch tests.patchSplinter Review
Attachment #8777791 - Flags: review?(lhansen)
Status: NEW → ASSIGNED
Attachment #8777791 - Flags: review?(lhansen) → review+
Pushed by bbouvier@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3e859924d2f5
Re-enable wasm test cases disabled; r=lth
https://hg.mozilla.org/mozilla-central/rev/3e859924d2f5
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.