Run jit-tests even if ENABLE_TRACEJIT isn't defined.

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Paul Biggar, Assigned: Paul Biggar)

Tracking

(Blocks: 1 bug)

Trunk
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixed-in-tracemonkey])

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 515727 [details] [diff] [review]
Test on all configs, not just ENABLE_TRACEJIT.

The jit-tests are predicated on ENABLE_TRACEJIT, though they work on all configurations I've tried. We should run them in all configs.
Attachment #515727 - Flags: review?(nnethercote)
Comment on attachment 515727 [details] [diff] [review]
Test on all configs, not just ENABLE_TRACEJIT.

Fine by me so long as it doesn't cause failures.  Though it looks like that patch is on top of some other changes in your tree?  Eg. you have JITFLAGS defined, which isn't in TM, but is in the patch I have in bug 635155, but I don't have the ',' at the start.
Attachment #515727 - Flags: review?(nnethercote) → review+
(Assignee)

Comment 2

7 years ago
Yup, it's on top of bug 637390 (which itself is based on bug 635155).
Oh, I should read all my bugmail before commenting... I see now this is on top of bug 637390.
(Assignee)

Comment 4

7 years ago
https://hg.mozilla.org/tracemonkey/rev/918ded339aa4
Whiteboard: [fixed-in-tracemonkey]
http://hg.mozilla.org/mozilla-central/rev/918ded339aa4
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.