1737798 - More memory64 test cases

Status: RESOLVED FIXED

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

Description

[Lars T Hansen [:lth]]

A bunch of things were not well tested for the initial landing, let's add more tests.

Comment 1

[Lars T Hansen [:lth]]

Attachment: Bug 1737798 - More memory64 test cases. r=yury

Test a number of corner cases for memory64. See comments in the tests
for further information.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/80065f480a1f
More memory64 test cases. r=yury

Comment 3

[Cristina Cozmuta (:CrissCozmuta)]

Backed out for causing TSAN exceptions. CLOSED TREE - as requested by the dev

Backout link : https://hg.mozilla.org/integration/autoland/rev/befddf7c6b413b684f5cda04f4878c71e84a4ddf

Push with failures : https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Crunnable&revision=80065f480a1f9e99814a89a8f601bad1ad5488a2

Comment 4

[Lars T Hansen [:lth]]

Yeah, these tests put a lot of pressure on the test systems due to very large heaps. That's unavoidable, so I'll need to figure out how to run these under controlled circumstances only. Locally I've had a lot of luck running with -j1, but I don't know yet how to tell that to the test systems.

Comment 5

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/80065f480a1f

Comment 6

[Lars T Hansen [:lth]]

Why was this closed? The patch was backed out, wasn't it?

Comment 7

[Cristian Tuns]

Backout merged to central: https://hg.mozilla.org/mozilla-central/rev/befddf7c6b41

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:lth, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/08ce6b6c9b40
More memory64 test cases. r=yury

Comment 10

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/08ce6b6c9b40