825276 - Inline the fast path of JS_WrapValue/JS_WrapObject

Status: NEW

(Core :: JavaScript Engine, defect)

Description

[Boris Zbarsky [:bzbarsky]]

Waldo thinks that us working around the lack of such inlining is nuts.  I'm glad he does!

Comment 1

[Boris Zbarsky [:bzbarsky]]

I just realized that given same-compartment wrappers I have no idea what the fast path should actually be.  :(

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: What I had before I realized that

Comment 3

[Bobby Holley (:bholley)]

Well, anything that's not an element or Components doesn't need a SCSW. Not sure how much that helps you.

FWIW, SCSWs aren't long for this earth. I'm close to killing SOWs, and then we can just hack something up for Components.

Comment 4

[Boris Zbarsky [:bzbarsky]]

> Not sure how much that helps you.

Not at all for JSAPI, since it has no idea about elements and such.

Is there a bug on killing SOWs I can mark this dependent on?

Comment 5

[Bobby Holley (:bholley)]

(In reply to Boris Zbarsky (:bz) from comment #4)
> Is there a bug on killing SOWs I can mark this dependent on?

Just filed one. bug 825392.