Closed
Bug 1199057
Opened 9 years ago
Closed 9 years ago
IonMonkey: MIPS32: Plumb new.target on the stack and make it accessible to JSNatives
Categories
(Core :: JavaScript Engine: JIT, defect)
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: hev, Assigned: hev)
References
Details
Attachments
(1 file)
3.10 KB,
patch
|
nbp
:
review+
efaust
:
feedback+
|
Details | Diff | Splinter Review |
python2 ./jit_test.py -f --tbpl /tmp/js.32 basic/newTargetRectifier.js Exit code: -10 FAIL - basic/newTargetRectifier.js [0|1|0|0] 16% ========> | 0.6s Exit code: -10 FAIL - basic/newTargetRectifier.js [0|2|0|0] 33% =================> | 0.6s Exit code: -10 FAIL - basic/newTargetRectifier.js [1|3|0|0] 66% ===================================> | 1.2s Exit code: -10 FAIL - basic/newTargetRectifier.js [1|4|0|0] 83% ============================================> | 1.2s Exit code: -10 FAIL - basic/newTargetRectifier.js [1|5|0|0] 100% ======================================================>| 1.4s
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → r
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8653224 -
Flags: review?(branislav.rankov)
Assignee | ||
Updated•9 years ago
|
Blocks: new.target
Assignee | ||
Updated•9 years ago
|
Attachment #8653224 -
Flags: review?(nicolas.b.pierron)
Comment 2•9 years ago
|
||
Comment on attachment 8653224 [details] [diff] [review] 0001-IonMonkey-MIPS32-Plumb-new.target-on-the-stack-and-m.patch Review of attachment 8653224 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/jit/mips32/Trampoline-mips32.cpp @@ +162,4 @@ > Loop over argv vector, push arguments onto stack in reverse order > ***************************************************************/ > > + // if we are constructing, that also needs to include newTarget nit: what is "that"?
Attachment #8653224 -
Flags: review?(nicolas.b.pierron)
Attachment #8653224 -
Flags: review+
Attachment #8653224 -
Flags: feedback?(efaustbmo)
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Attachment #8653224 -
Flags: feedback?(efaustbmo) → feedback+
https://hg.mozilla.org/mozilla-central/rev/8289ad7fa005
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•