wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Backing out patch for Bug 489623 due to breaking v8-bench.

Browse Source
This commit is contained in:
Kannan Vijayan 2012-04-27 15:29:29 -04:00
parent 6682595eb7
commit c7e2c9a499
1 changed files with 1 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
js/src/ion/IonCaches.cpp
View File

@ -456,10 +456,6 @@ IsEligibleForInlinePropertyAdd(JSContext *cx, JSObject *obj, jsid propId, uint32
if (!propShape || propShape->inDictionary() || !propShape->hasSlot() || !propShape->hasDefaultSetter())
return false;
// If object has a non-default resolve hook, don't inline
if (obj->getClass()->resolve != JS_ResolveStub)
return false;
// walk up the object prototype chain and ensure that all prototypes
// are native, and that all prototypes have no getter or setter
// defined on the property
@ -470,17 +466,7 @@ IsEligibleForInlinePropertyAdd(JSContext *cx, JSObject *obj, jsid propId, uint32
// if prototype defines this property in a non-plain way, don't optimize
const Shape *protoShape = proto->nativeLookup(cx, propId);
if (protoShape) {
// If the prototype has a property with this name, we may be able to
// inline-add if the property has a default setter. If there's no
// default setter, then setting would potentially cause special
// behaviour, so we choose not to optimize.
return protoShape->hasDefaultSetter();
}
// Otherise, if there's no such property, watch out for a resolve hook that would need
// to be invoked and thus prevent inlining of property addition.
if (proto->getClass()->resolve != JS_ResolveStub)
if (protoShape && !protoShape->hasDefaultSetter())
return false;
}