Closed Bug 863858 Opened 7 years ago Closed 7 years ago

Allow initial heap placement from more places

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch v0 (obsolete) — Splinter Review
Brian found many places in Bug 851057 where we were not passing through the initial heap correctly to the underlying object creation routines.
Attachment #739776 - Flags: review?(wmccloskey)
Attachment #739776 - Flags: review?(terrence)
Attachment #739776 - Flags: feedback?(bhackett1024)
Attachment #739776 - Flags: review?(terrence) → review+
Comment on attachment 739776 [details] [diff] [review]
v0

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

I don't know enough to review this code. Please ask dvander if you want additional review.
Attachment #739776 - Flags: review?(wmccloskey)
The changes here are straightforward, I'd just like a quick sanity check on the changes to our initial heap placement when running in JIT code.
Attachment #739776 - Attachment is obsolete: true
Attachment #739776 - Flags: feedback?(bhackett1024)
Attachment #744328 - Flags: review?(dvander)
Attachment #744328 - Flags: review?(dvander) → review+
https://hg.mozilla.org/mozilla-central/rev/776c624e358c
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.