--jitflags=none is broken in jit_test.py

RESOLVED FIXED in Firefox 39

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: arai, Assigned: arai)

Tracking

Trunk
mozilla40
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox39 fixed, firefox40 fixed, firefox-esr38 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

4 years ago
jit_test.py with --jitflags=none or without any jitflags related option runs only asm.js tests.

JITFLAGS['none'] should contain an empty list for jit_test.py.
Assignee

Comment 1

4 years ago
Added empty list in JITFLAGS['none'] in tests.py, for jit_test.py,

Also, in that case, it should be better to return None for jstests.py, so added keyword arguments to override the return value for 'none' case.

This also needs aurora/esr38 uplift, since patches in bug 1161410 were uplifted.
Assignee: nobody → arai.unmht
Attachment #8603404 - Flags: review?(sphink)
Assignee

Comment 2

4 years ago
Pushed to try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a1fb35b98156
--jitflags=none is not used in automation tho, just to confirm no other thing is broken by this change.
Comment on attachment 8603404 [details] [diff] [review]
Fix --jitflags=none or no --jitflags in jit_test.py.

Review of attachment 8603404 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM! Thanks for the fix!
Attachment #8603404 - Flags: review?(sphink) → review+
Assignee

Comment 5

4 years ago
Comment on attachment 8603404 [details] [diff] [review]
Fix --jitflags=none or no --jitflags in jit_test.py.

for aurora
Approval Request Comment
[Feature/regressing bug #]: bug 1161410
[User impact if declined]: no user impact, but automated test (jit_test.py) doesn't work correctly, and most tests are skipped on local run (no impact for treeherder)
[Describe test coverage new/current, TreeHerder]: now mozilla-inbound test is running https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=d732da9298fa
[Risks and why]: Low, since it's test-only (test harness bug fix)
[String/UUID change made/needed]: None

for esr38
[Approval Request Comment]
If this is not a sec:{high,crit} bug, please state case for ESR consideration: buggy patch was uplifted there in bug 1161410
User impact if declined: no user impact, but automated test (jit_test.py) doesn't work correctly, and most tests are skipped on local run (no impact for treeherder)
Fix Landed on Version: landed to mozilla-inbound, which will be firefox40
Risk to taking this patch (and alternatives if risky):  Low, since it's test-only (test harness bug fix)
String or UUID changes made by this patch: None
Attachment #8603404 - Flags: approval-mozilla-esr38?
Attachment #8603404 - Flags: approval-mozilla-aurora?
Comment on attachment 8603404 [details] [diff] [review]
Fix --jitflags=none or no --jitflags in jit_test.py.

These can land with a=test-only
Attachment #8603404 - Flags: approval-mozilla-esr38?
Attachment #8603404 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/d732da9298fa
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.