Last Comment Bug 774510 - Remove uncessary typeOf hooks
: Remove uncessary typeOf hooks
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: All All
-- normal (vote)
: ---
Assigned To: Tom Schuster [:evilpie]
: Jason Orendorff [:jorendorff]
Depends on:
Blocks: 725966
  Show dependency treegraph
Reported: 2012-07-16 16:07 PDT by Tom Schuster [:evilpie]
Modified: 2012-07-20 16:37 PDT (History)
1 user (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

v1 (5.87 KB, patch)
2012-07-16 16:07 PDT, Tom Schuster [:evilpie]
no flags Details | Diff | Splinter Review
v1 (5.87 KB, patch)
2012-07-16 16:09 PDT, Tom Schuster [:evilpie]
dvander: review+
Details | Diff | Splinter Review

Description User image Tom Schuster [:evilpie] 2012-07-16 16:07:19 PDT
Created attachment 642776 [details] [diff] [review]

The default behaviour for objects without a typeOf hook is to return "object". In the most cases we actually want that and don't need to implement that hook and explicitly return JSTYPE_OBJECT. This is mostly done to prevent bailouts for MTypeOfIs on dense arrays in IonMonkey, which are probably common.
Comment 1 User image Tom Schuster [:evilpie] 2012-07-16 16:09:57 PDT
Created attachment 642778 [details] [diff] [review]

Forgot to refresh.
Comment 2 User image Tom Schuster [:evilpie] 2012-07-18 04:53:07 PDT
Comment 3 User image Tom Schuster [:evilpie] 2012-07-18 04:53:27 PDT
We could also land this on m-i.

Note You need to log in before you can comment on or make changes to this bug.