Bug 18643 - SQUIRRELFISH: need to support implicit function calls (valueOf, toString)
Summary: SQUIRRELFISH: need to support implicit function calls (valueOf, toString)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Oliver Hunt
URL:
Keywords:
Depends on:
Blocks: 18624
  Show dependency treegraph
 
Reported: 2008-04-20 16:36 PDT by Maciej Stachowiak
Modified: 2008-04-29 22:15 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Stachowiak 2008-04-20 16:36:14 PDT
SquirrelFish does not yet properly handle property getters and setters.
Comment 1 Oliver Hunt 2008-04-23 13:14:03 PDT
The problem for these implicit function calls (valueOf is called by toNumber, toString by +, etc, getters/setters by resolve) is that they may require a function call in the VM, which may inturn result in resizing of the RegisterFile which would invalidate state in the originating privateExecute frame.  Adding the required guards around all call sites would almost certainly be a perf hit but maybe possible at a few places (resolve?)

Comment 2 Cameron Zwarich (cpst) 2008-04-23 21:20:47 PDT
This is causing a lot of failures in the remaining JavaScriptCore test failures, e.g. the instanceof tests. We should really find a way to fix this without too large of a performance regression.
Comment 3 Maciej Stachowiak 2008-04-24 00:50:33 PDT
I'm not sure valueOf / toString are quire the same issue as getters and setters, and we might not want to solve them the exact same way necessarily.
Comment 4 Oliver Hunt 2008-04-29 22:15:40 PDT
Committed r32669 and r32718