1353384 - Optimize GetName TypeOfNoProperty

Status: NEW

(Core :: JavaScript Engine: JIT, enhancement, P3)

Description

[Jan de Mooij [:jandem]]

If we have |typeof x| and x does not exist, we attach an IC stub that guards on the environment objects but it also has to guard the global's proto chain.

Unfortunately this is annoying because in the browser we have the GSP proxy on the prototype chain and we would need some way to guard it doesn't have the property.

We should probably fix this before converting this code to CacheIR (bug 1326437) as it will also affect Baseline then.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Testcase

This prints "Fail" (I get "Pass" with Ion disabled).

Comment 2

[Jan de Mooij [:jandem]]

Boris, any thoughts on how we can deal with the GSP/WindowNamedPropertiesHandler proxy? Can we do something similar to the other DOM proxies?

Comment 3

[Boris Zbarsky [:bzbarsky]]

Hmm.  So this one is different from other DOM proxies in various ways.  It has different slot layout, no expando, etc.

But I guess the real question is whether we could give it a concept of "shape" or "generation" or something like that, right?  Something that the JIT could then guard on.  It seems like we should be able to do that, generally speaking, either as an ExpandoAndGeneration pointer (with an undefined expando in all cases) or a direct counter in a PrivateValue (which would presumably need to count by 2 to make sense in that case, right?).  I assume we do want this thing somewhere where the jit can reach it via slots.

We'd also need the JIT to recognize this specific proxy handler, I guess.

Comment 4

[Jan de Mooij [:jandem]]

@bz: thanks. Yes we need a generation field for this I suppose... We could then also use this to optimize |window.missingProp|. It would be nice if we could share the ExpandoAndGeneration code.

Tom and I talked about this yesterday and for now it seems best to remove this (buggy) optimization as part of the CacheIR conversion in bug 1326437. That unblocks bug 1326437 and we can then use this bug to add it back with the correct guards.