Ignore:
Timestamp:
Sep 24, 2008, 10:02:14 AM (17 years ago)
Author:
Darin Adler
Message:

2008-09-24 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

About 1% on v8-raytrace.

  • kjs/JSActivation.cpp: Moved copyRegisters to the header to make it inline.
  • kjs/JSActivation.h: (JSC::JSActivation::copyRegisters): Moved here. Also removed the registerArraySize argument to setRegisters, since the object doesn't need to store the number of registers.
  • kjs/JSGlobalObject.cpp: (JSC::JSGlobalObject::reset): Removed unnecessary clearing left over from when we used this on objects that weren't brand new. These days, this function is really just part of the constructor.
  • kjs/JSGlobalObject.h: Added registerArraySize to JSGlobalObjectData, since JSVariableObjectData no longer needs it. Added a setRegisters override here that handles storing the size.
  • kjs/JSStaticScopeObject.h: Removed code to set registerArraySize, since it no longer exists.
  • kjs/JSVariableObject.cpp: Moved copyRegisterArray and setRegisters to the header to make them inline.
  • kjs/JSVariableObject.h: Removed registerArraySize from JSVariableObjectData, since it was only used for the global object. (JSC::JSVariableObject::copyRegisterArray): Moved here ot make it inline. (JSC::JSVariableObject::setRegisters): Moved here to make it inline. Also removed the code to set registerArraySize and changed an if statement into an assert to save an unnnecessary branch.
File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/JavaScriptCore/kjs/JSVariableObject.cpp

    r36821 r36851  
    6969}
    7070
    71 Register* JSVariableObject::copyRegisterArray(Register* src, size_t count)
    72 {
    73     Register* registerArray = new Register[count];
    74     memcpy(registerArray, src, count * sizeof(Register));
    75 
    76     return registerArray;
    77 }
    78 
    79 void JSVariableObject::setRegisters(Register* r, Register* registerArray, size_t count)
    80 {
    81     if (registerArray != d->registerArray.get())
    82         d->registerArray.set(registerArray);
    83     d->registerArraySize = count;
    84     d->registers = r;
    85 }
    86 
    8771} // namespace JSC
Note: See TracChangeset for help on using the changeset viewer.