Bug 1010775 part 5 - Pass ImmGCPtr and ImmMaybeNurseryPtr by value instead of const-ref. r=sunfish

This commit is contained in:
Jan de Mooij 2014-05-15 17:42:38 +02:00
parent 97feec9143
commit 042e72a91d
10 changed files with 20 additions and 20 deletions

View File

@ -244,7 +244,7 @@ template void MacroAssembler::guardType(const ValueOperand &value, types::Type t
Register scratch, Label *miss);
void
MacroAssembler::branchNurseryPtr(Condition cond, const Address &ptr1, const ImmMaybeNurseryPtr &ptr2,
MacroAssembler::branchNurseryPtr(Condition cond, const Address &ptr1, ImmMaybeNurseryPtr ptr2,
Label *label)
{
#ifdef JSGC_GENERATIONAL
@ -255,7 +255,7 @@ MacroAssembler::branchNurseryPtr(Condition cond, const Address &ptr1, const ImmM
}
void
MacroAssembler::moveNurseryPtr(const ImmMaybeNurseryPtr &ptr, Register reg)
MacroAssembler::moveNurseryPtr(ImmMaybeNurseryPtr ptr, Register reg)
{
#ifdef JSGC_GENERATIONAL
if (ptr.value && gc::IsInsideNursery(GetIonContext()->cx->runtime(), (void *)ptr.value))

View File

@ -692,9 +692,9 @@ class MacroAssembler : public MacroAssemblerSpecific
bind(&done);
}
void branchNurseryPtr(Condition cond, const Address &ptr1, const ImmMaybeNurseryPtr &ptr2,
void branchNurseryPtr(Condition cond, const Address &ptr1, ImmMaybeNurseryPtr ptr2,
Label *label);
void moveNurseryPtr(const ImmMaybeNurseryPtr &ptr, Register reg);
void moveNurseryPtr(ImmMaybeNurseryPtr ptr, Register reg);
void canonicalizeDouble(FloatRegister reg) {
Label notNaN;

View File

@ -1329,7 +1329,7 @@ class Assembler : public AssemblerShared
// As opposed to x86/x64 version, the data relocation has to be executed
// before to recover the pointer, and not after.
void writeDataRelocation(const ImmGCPtr &ptr) {
void writeDataRelocation(ImmGCPtr ptr) {
if (ptr.value)
tmpDataRelocations_.append(nextOffset());
}

View File

@ -479,7 +479,7 @@ MacroAssemblerARM::ma_mov(ImmWord imm, Register dest,
}
void
MacroAssemblerARM::ma_mov(const ImmGCPtr &ptr, Register dest)
MacroAssemblerARM::ma_mov(ImmGCPtr ptr, Register dest)
{
// As opposed to x86/x64 version, the data relocation has to be executed
// before to recover the pointer, and not after.
@ -2023,7 +2023,7 @@ MacroAssemblerARMCompat::movePtr(ImmWord imm, Register dest)
ma_mov(Imm32(imm.value), dest);
}
void
MacroAssemblerARMCompat::movePtr(const ImmGCPtr &imm, Register dest)
MacroAssemblerARMCompat::movePtr(ImmGCPtr imm, Register dest)
{
ma_mov(imm, dest);
}
@ -2513,7 +2513,7 @@ MacroAssemblerARMCompat::cmpPtr(Register lhs, Register rhs)
}
void
MacroAssemblerARMCompat::cmpPtr(Register lhs, const ImmGCPtr &rhs)
MacroAssemblerARMCompat::cmpPtr(Register lhs, ImmGCPtr rhs)
{
ma_cmp(lhs, rhs);
}

View File

@ -120,7 +120,7 @@ class MacroAssemblerARM : public Assembler
void ma_mov(ImmWord imm, Register dest,
SetCond_ sc = NoSetCond, Condition c = Always);
void ma_mov(const ImmGCPtr &ptr, Register dest);
void ma_mov(ImmGCPtr ptr, Register dest);
// Shifts (just a move with a shifting op2)
void ma_lsl(Imm32 shift, Register src, Register dst);
@ -1315,7 +1315,7 @@ class MacroAssemblerARMCompat : public MacroAssemblerARM
void movePtr(ImmWord imm, Register dest);
void movePtr(ImmPtr imm, Register dest);
void movePtr(const AsmJSImmPtr &imm, Register dest);
void movePtr(const ImmGCPtr &imm, Register dest);
void movePtr(ImmGCPtr imm, Register dest);
void load8SignExtend(const Address &address, Register dest);
void load8SignExtend(const BaseIndex &src, Register dest);
@ -1414,7 +1414,7 @@ class MacroAssemblerARMCompat : public MacroAssemblerARM
void cmpPtr(Register lhs, ImmWord rhs);
void cmpPtr(Register lhs, ImmPtr rhs);
void cmpPtr(Register lhs, Register rhs);
void cmpPtr(Register lhs, const ImmGCPtr &rhs);
void cmpPtr(Register lhs, ImmGCPtr rhs);
void cmpPtr(Register lhs, Imm32 rhs);
void cmpPtr(const Address &lhs, Register rhs);
void cmpPtr(const Address &lhs, ImmWord rhs);

View File

@ -732,7 +732,7 @@ class Assembler : public AssemblerShared
// As opposed to x86/x64 version, the data relocation has to be executed
// before to recover the pointer, and not after.
void writeDataRelocation(const ImmGCPtr &ptr) {
void writeDataRelocation(ImmGCPtr ptr) {
if (ptr.value)
dataRelocations_.writeUnsigned(nextOffset().getOffset());
}

View File

@ -247,7 +247,7 @@ MacroAssemblerMIPS::ma_move(Register rd, Register rs)
}
void
MacroAssemblerMIPS::ma_li(Register dest, const ImmGCPtr &ptr)
MacroAssemblerMIPS::ma_li(Register dest, ImmGCPtr ptr)
{
writeDataRelocation(ptr);
ma_liPatchable(dest, Imm32(ptr.value));
@ -1762,7 +1762,7 @@ MacroAssemblerMIPSCompat::movePtr(ImmWord imm, Register dest)
}
void
MacroAssemblerMIPSCompat::movePtr(const ImmGCPtr &imm, Register dest)
MacroAssemblerMIPSCompat::movePtr(ImmGCPtr imm, Register dest)
{
ma_li(dest, imm);
}

View File

@ -103,7 +103,7 @@ class MacroAssemblerMIPS : public Assembler
void ma_move(Register rd, Register rs);
void ma_li(Register dest, const ImmGCPtr &ptr);
void ma_li(Register dest, ImmGCPtr ptr);
void ma_li(Register dest, AbsoluteLabel *label);
@ -1027,7 +1027,7 @@ public:
void movePtr(ImmWord imm, Register dest);
void movePtr(ImmPtr imm, Register dest);
void movePtr(const AsmJSImmPtr &imm, Register dest);
void movePtr(const ImmGCPtr &imm, Register dest);
void movePtr(ImmGCPtr imm, Register dest);
void load8SignExtend(const Address &address, Register dest);
void load8SignExtend(const BaseIndex &src, Register dest);

View File

@ -142,7 +142,7 @@ class AssemblerX86Shared : public AssemblerShared
dataRelocations_.writeUnsigned(masm.currentOffset());
}
}
void writeDataRelocation(const ImmGCPtr &ptr) {
void writeDataRelocation(ImmGCPtr ptr) {
if (ptr.value)
dataRelocations_.writeUnsigned(masm.currentOffset());
}

View File

@ -185,7 +185,7 @@ class Assembler : public AssemblerX86Shared
// Actual assembly emitting functions.
void push(const ImmGCPtr &ptr) {
void push(ImmGCPtr ptr) {
push(Imm32(ptr.value));
writeDataRelocation(ptr);
}
@ -218,11 +218,11 @@ class Assembler : public AssemblerX86Shared
return movWithPatch(ImmWord(uintptr_t(imm.value)), dest);
}
void movl(const ImmGCPtr &ptr, Register dest) {
void movl(ImmGCPtr ptr, Register dest) {
masm.movl_i32r(ptr.value, dest.code());
writeDataRelocation(ptr);
}
void movl(const ImmGCPtr &ptr, const Operand &dest) {
void movl(ImmGCPtr ptr, const Operand &dest) {
switch (dest.kind()) {
case Operand::REG:
masm.movl_i32r(ptr.value, dest.reg());