Closed
Bug 697282
Opened 13 years ago
Closed 13 years ago
IonMonkey: Support JSOP_CALLPROP
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: masterofhats, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 1 obsolete file)
183 bytes,
text/plain
|
Details | |
901 bytes,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; InfoPath.2; .NET4.0C; .NET4.0E; MS-RTC LM 8) Steps to reproduce: I executed the attached js, which essentially calls an object's method, using the js shell: js --ion --ion-eager mehtod.js Actual results: I got [MIR] Analying script js-samples/method.js:8 [Abort] Unsupported opcode: callprop (line 9) Expected results: IonMonkey should have compiled the function and the contained method call.
Comment 1•13 years ago
|
||
This is expected behavior, since IonMonkey is still in the early stages, and there are many things we haven't yet implemented. We don't even support calling global functions yet (bug 694169). This also depends on us supporting JSOP_GETPROP. I've renamed the bug so that we can get back to it once we're at that point.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: IonMonkey seems not to support method calls → IonMonkey: Support JSOP_CALLPROP
Comment 2•13 years ago
|
||
With Bug 712714 resolved in-tree, JSOP_CALLPROP is the same as JSOP_GETPROP.
Attachment #587510 -
Flags: review?(dvander)
Updated•13 years ago
|
Attachment #587510 -
Flags: review?(dvander) → review+
Comment 3•13 years ago
|
||
JSOP_SWAP needs to be followed by JSOP_NOTEARG in the context of JSOP_CALLPROP.
Attachment #587510 -
Attachment is obsolete: true
Attachment #587522 -
Flags: review?(dvander)
Updated•13 years ago
|
Attachment #587522 -
Flags: review?(dvander) → review+
Comment 4•13 years ago
|
||
http://hg.mozilla.org/projects/ionmonkey/rev/d601339c1bea
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•