Closed Bug 866916 Opened 11 years ago Closed 11 years ago

JavaScript JS_MORE_DETERMINISTIC builds are busted

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
blocker

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: gkw, Assigned: gkw)

References

Details

(Keywords: regression)

Attachments

(2 files, 1 obsolete file)

Attached file stack (deleted) —
JavaScript JS_MORE_DETERMINISTIC builds are busted, see stack. It wasn't broken last week.
Double confirmed:

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   130017:9ab1119d4596
user:        Nicholas Nethercote
date:        Tue Apr 23 23:44:36 2013 -0700
summary:     Bug 634839 (part 1) - Remove unnecessary headers (as found by include-what-you-use) from js/src/jsfoo.cpp files.  r=luke.
Attached patch patch v1 (obsolete) (deleted) — Splinter Review
Assignee: general → gary
Status: NEW → ASSIGNED
Attachment #743303 - Flags: review?(sphink)
Removed the ifdef as recommended by sfink in-person, ifdefs are ugly.
Attachment #743303 - Attachment is obsolete: true
Attachment #743303 - Flags: review?(sphink)
Comment on attachment 743305 [details] [diff] [review]
patch v2 - ifdefs are unnecessary

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

Sorry for the delay. Rebuilding my computer.
Attachment #743305 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/e963546ec749
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: