Remove tracer-specific stuff from jit-tests

RESOLVED FIXED in mozilla11

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: dmandelin, Assigned: dmandelin)

Tracking

Trunk
mozilla11
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
Created attachment 578758 [details] [diff] [review]
WIP

There is TMFLAGS support in the harness, and checkStats in the test cases.

Posting a WIP that removes those things but isn't done. I removed HOTLOOP and RUNLOOP too, but there are lots of tests that use those, so we need to put them back, modify the tests, or delete the tests.

I removed some tests that only checked checkStats stuff, or were otherwise totally tracer-specific. I suspect that many of the other tests that used to call checkStats, and some other tests, are pretty much specific to the tracer and are unlikely to catch many bugs now.
(Assignee)

Comment 1

6 years ago
Created attachment 579149 [details] [diff] [review]
Patch
Assignee: general → dmandelin
Attachment #578758 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #579149 - Flags: review?(dvander)
Attachment #579149 - Flags: review?(dvander) → review+
(Assignee)

Comment 2

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/39b7b2f8e840

Comment 3

6 years ago
https://hg.mozilla.org/mozilla-central/rev/39b7b2f8e840
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.