Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-08 04:27:37 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[INFER] Use initialized length instead of capacity in Array.join optimization, bug 655769.

Browse Source
This commit is contained in:
Brian Hackett 2011-05-14 07:39:42 -07:00
parent a04ccaea1a
commit 8b3d4aaadf
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
js/src/jsarray.cpp
View File

@ -1405,9 +1405,9 @@ array_toString_sub(JSContext *cx, JSObject *obj, JSBool locale,
StringBuffer sb(cx);
if (!locale && !seplen && obj->isDenseArray() && !js_PrototypeHasIndexedProperties(cx, obj)) {
/* Elements beyond 'capacity' are 'undefined' and thus can be ignored. */
/* Elements beyond the initialized length are 'undefined' and thus can be ignored. */
Value *beg = obj->getDenseArrayElements();
Value *end = beg + Min(length, obj->getDenseArrayCapacity());
Value *end = beg + Min(length, obj->getDenseArrayInitializedLength());
for (Value *vp = beg; vp != end; ++vp) {
if (!JS_CHECK_OPERATION_LIMIT(cx))
return false;

4
js/src/methodjit/Compiler.cpp
View File

@ -2200,7 +2200,7 @@ mjit::Compiler::generateMethod()
*/
if (cx->typeInferenceEnabled()) {
uint32 slot = ArgSlot(GET_SLOTNO(PC));
if (a->varTypes[slot].type == JSVAL_TYPE_DOUBLE && analysis->trackSlot(slot))
if (analysis->trackSlot(slot) && a->varTypes[slot].type == JSVAL_TYPE_DOUBLE)
frame.ensureDouble(frame.getArg(GET_SLOTNO(PC)));
}
@ -2228,7 +2228,7 @@ mjit::Compiler::generateMethod()
if (cx->typeInferenceEnabled()) {
uint32 slot = LocalSlot(script, GET_SLOTNO(PC));
if (a->varTypes[slot].type == JSVAL_TYPE_DOUBLE && analysis->trackSlot(slot))
if (analysis->trackSlot(slot) && a->varTypes[slot].type == JSVAL_TYPE_DOUBLE)
frame.ensureDouble(frame.getLocal(GET_SLOTNO(PC)));
}