Closed
Bug 656059
Opened 13 years ago
Closed 13 years ago
Remove TraceRecorder::getClassPrototype(JSProtoKey, LIns*&), replace with TR::getObjectPrototype(LIns*&) and similar
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla10
People
(Reporter: Waldo, Assigned: Waldo)
Details
Attachments
(2 files)
489 bytes,
patch
|
cjones
:
review+
|
Details | Diff | Splinter Review |
7.81 KB,
patch
|
jorendorff
:
review+
|
Details | Diff | Splinter Review |
If you want a class prototype, you should ask for it specifically. Asking specifically gives us more freedom in how we store prototypes (and constructors, for now) in the global object, and it's cleaner and more readable than having to know what a |key| variable could be when it's used in a centralized method. There are a few different methods for getting an original prototype in use in jstracer.cpp; this bug will fix the one that accepts a JSProtoKey to determine what prototype to get.
Assignee | ||
Comment 1•13 years ago
|
||
It turns out that DebugOnly<T> isn't so hot when T is a pointer type right now.
Attachment #531374 -
Flags: review?(jones.chris.g)
Assignee | ||
Comment 2•13 years ago
|
||
Attachment #531375 -
Flags: review?(jorendorff)
Updated•13 years ago
|
Attachment #531374 -
Flags: review?(jones.chris.g) → review+
Assignee | ||
Comment 3•13 years ago
|
||
http://hg.mozilla.org/tracemonkey/rev/c2315c604e7c for DebugOnly<T>::operator->, but the main patch here still remains to be reviewed/committed.
Comment 4•13 years ago
|
||
cdleary-bot mozilla-central merge info: http://hg.mozilla.org/mozilla-central/rev/c2315c604e7c Note: not marking as fixed because fixed-in-tracemonkey is not present on the whiteboard.
Comment 5•13 years ago
|
||
Comment on attachment 531375 [details] [diff] [review] Ask for specific prototypes, not for a prototype specified key-wise Sure, ok.
Attachment #531375 -
Flags: review?(jorendorff) → review+
Assignee | ||
Comment 6•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d5e6ea418eea
Target Milestone: --- → mozilla10
Assignee | ||
Comment 7•13 years ago
|
||
And this, to fix compiler warnings that gcc 4.6.1 is apparently smart enough (or conservative enough) to not emit: https://hg.mozilla.org/integration/mozilla-inbound/rev/e13c356c0a45
Comment 8•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d5e6ea418eea https://hg.mozilla.org/mozilla-central/rev/e13c356c0a45
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•