Merge pull request #2522 from lioncash/ref

Core: Pass several parameters by const reference
This commit is contained in:
comex 2015-06-04 16:15:11 -04:00
commit 0c23283049
9 changed files with 16 additions and 16 deletions

View file

@ -414,7 +414,7 @@ static int GetVal(const char *flt, char chr)
return ret;
}
static int alphatobin(u32 *dst, std::vector<std::string> alpha, int size)
static int alphatobin(u32 *dst, const std::vector<std::string>& alpha, int size)
{
int j = 0;
int ret = 0;

View file

@ -272,11 +272,11 @@ private:
void round_long_acc(Gen::X64Reg long_acc = Gen::EAX);
void set_long_acc(int _reg, Gen::X64Reg acc = Gen::EAX);
void get_acc_h(int _reg, Gen::X64Reg acc = Gen::EAX, bool sign = true);
void set_acc_h(int _reg, Gen::OpArg arg = R(Gen::EAX));
void set_acc_h(int _reg, const Gen::OpArg& arg = R(Gen::EAX));
void get_acc_m(int _reg, Gen::X64Reg acc = Gen::EAX, bool sign = true);
void set_acc_m(int _reg, Gen::OpArg arg = R(Gen::EAX));
void set_acc_m(int _reg, const Gen::OpArg& arg = R(Gen::EAX));
void get_acc_l(int _reg, Gen::X64Reg acc = Gen::EAX, bool sign = true);
void set_acc_l(int _reg, Gen::OpArg arg = R(Gen::EAX));
void set_acc_l(int _reg, const Gen::OpArg& arg = R(Gen::EAX));
void get_long_acx(int _reg, Gen::X64Reg acx = Gen::EAX);
void get_ax_l(int _reg, Gen::X64Reg acx = Gen::EAX);
void get_ax_h(int _reg, Gen::X64Reg acc = Gen::EAX);

View file

@ -763,7 +763,7 @@ void DSPEmitter::get_acc_l(int _reg, X64Reg acl, bool sign)
gpr.readReg(_reg+DSP_REG_ACL0, acl, sign?SIGN:ZERO);
}
void DSPEmitter::set_acc_l(int _reg, OpArg arg)
void DSPEmitter::set_acc_l(int _reg, const OpArg& arg)
{
// return g_dsp.r[DSP_REG_ACM0 + _reg];
gpr.writeReg(_reg+DSP_REG_ACL0,arg);
@ -777,7 +777,7 @@ void DSPEmitter::get_acc_m(int _reg, X64Reg acm, bool sign)
}
// In: s16 in AX
void DSPEmitter::set_acc_m(int _reg, OpArg arg)
void DSPEmitter::set_acc_m(int _reg, const OpArg& arg)
{
// return g_dsp.r.ac[_reg].m;
gpr.writeReg(_reg+DSP_REG_ACM0,arg);
@ -791,7 +791,7 @@ void DSPEmitter::get_acc_h(int _reg, X64Reg ach, bool sign)
}
// In: s16 in AX
void DSPEmitter::set_acc_h(int _reg, OpArg arg)
void DSPEmitter::set_acc_h(int _reg, const OpArg& arg)
{
// return g_dsp.r[DSP_REG_ACM0 + _reg];
gpr.writeReg(_reg+DSP_REG_ACH0,arg);

View file

@ -33,7 +33,7 @@ bool GeckoCode::Exist(u32 address, u32 data) const
}
// return true if the code is identical
bool GeckoCode::Compare(GeckoCode compare) const
bool GeckoCode::Compare(const GeckoCode& compare) const
{
if (codes.size() != compare.codes.size())
return false;

View file

@ -36,7 +36,7 @@ namespace Gecko
bool enabled;
bool user_defined;
bool Compare(GeckoCode compare) const;
bool Compare(const GeckoCode& compare) const;
bool Exist(u32 address, u32 data) const;
};

View file

@ -347,7 +347,7 @@ u16 GCMemcard::GetFreeBlocks() const
return BE16(CurrentBat->FreeBlocks);
}
u8 GCMemcard::TitlePresent(DEntry d) const
u8 GCMemcard::TitlePresent(const DEntry& d) const
{
if (!m_valid)
return DIRLEN;

View file

@ -360,7 +360,7 @@ public:
u16 GetFreeBlocks() const;
// If title already on memcard returns index, otherwise returns -1
u8 TitlePresent(DEntry d) const;
u8 TitlePresent(const DEntry& d) const;
bool GCI_FileName(u8 index, std::string &filename) const;
// DEntry functions, all take u8 index < DIRLEN (127)

View file

@ -347,7 +347,7 @@ void GPRRegCache::LoadRegister(size_t preg, X64Reg newLoc)
emit->MOV(32, ::Gen::R(newLoc), regs[preg].location);
}
void GPRRegCache::StoreRegister(size_t preg, OpArg newLoc)
void GPRRegCache::StoreRegister(size_t preg, const OpArg& newLoc)
{
emit->MOV(32, newLoc, regs[preg].location);
}
@ -357,7 +357,7 @@ void FPURegCache::LoadRegister(size_t preg, X64Reg newLoc)
emit->MOVAPD(newLoc, regs[preg].location);
}
void FPURegCache::StoreRegister(size_t preg, OpArg newLoc)
void FPURegCache::StoreRegister(size_t preg, const OpArg& newLoc)
{
emit->MOVAPD(newLoc, regs[preg].location.GetSimpleReg());
}

View file

@ -90,7 +90,7 @@ public:
//read only will not set dirty flag
void BindToRegister(size_t preg, bool doLoad = true, bool makeDirty = true);
void StoreFromRegister(size_t preg, FlushMode mode = FLUSH_ALL);
virtual void StoreRegister(size_t preg, Gen::OpArg newLoc) = 0;
virtual void StoreRegister(size_t preg, const Gen::OpArg& newLoc) = 0;
virtual void LoadRegister(size_t preg, Gen::X64Reg newLoc) = 0;
const Gen::OpArg &R(size_t preg) const
@ -159,7 +159,7 @@ public:
class GPRRegCache : public RegCache
{
public:
void StoreRegister(size_t preg, Gen::OpArg newLoc) override;
void StoreRegister(size_t preg, const Gen::OpArg& newLoc) override;
void LoadRegister(size_t preg, Gen::X64Reg newLoc) override;
Gen::OpArg GetDefaultLocation(size_t reg) const override;
const int* GetAllocationOrder(size_t& count) override;
@ -172,7 +172,7 @@ public:
class FPURegCache : public RegCache
{
public:
void StoreRegister(size_t preg, Gen::OpArg newLoc) override;
void StoreRegister(size_t preg, const Gen::OpArg& newLoc) override;
void LoadRegister(size_t preg, Gen::X64Reg newLoc) override;
const int* GetAllocationOrder(size_t& count) override;
Gen::OpArg GetDefaultLocation(size_t reg) const override;