This failure is very frequent on Windows 7 opt and pgo. Failures began with bug 1355573: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-searchStr=windows%20jit&tochange=fe64a3b770fd421ee62896bce58369275555281b&fromchange=4a50cfa3de8009f8e6900ce44a53afa45e28428e
Whiteboard: [stockwell needswork]
I added this test a few days ago. It allocates some huge strings and apparently that can make us OOM on 32-bit Windows. I think we can just add allow-oom to the test.
Assignee: nobody → jdemooij
Status: NEW → ASSIGNED
Attachment #8873394 - Flags: review?(jcoppeard)
Attachment #8873394 - Flags: review?(jcoppeard) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/0b3be4de7ac3 Add allow-oom attribute to bug1355573.js jit-test. r=jonco
Hm this doesn't work because the test harness first checks the expected error and so it never gets to the "allow oom" part...
Another patch to avoid using both "error" and "allow-oom". It's tempting to just fix the harness but I don't have the time for it now.
Attachment #8873755 - Flags: review?(jcoppeard)
Attachment #8873755 - Flags: review?(jcoppeard) → review+
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/0678eef8a8bd part 2 - Rewrite bug1355573.js jit-test to work around the harness not supporting error + allow-oom. r=jonco
Status: REOPENED → RESOLVED
Last Resolved: 2 years ago → 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.