Merge pull request #609 from Tilka/fix_warnings
Fix more warnings from #579
This commit is contained in:
commit
601a406e95
|
@ -1011,7 +1011,7 @@ void DSPJitRegCache::spillXReg(X64Reg reg)
|
||||||
if (xregs[reg].guest_reg <= DSP_REG_MAX_MEM_BACKED)
|
if (xregs[reg].guest_reg <= DSP_REG_MAX_MEM_BACKED)
|
||||||
{
|
{
|
||||||
_assert_msg_(DSPLLE, !regs[xregs[reg].guest_reg].used,
|
_assert_msg_(DSPLLE, !regs[xregs[reg].guest_reg].used,
|
||||||
"to be spilled host reg %x(guest reg %x) still in use!",
|
"to be spilled host reg %x(guest reg %zx) still in use!",
|
||||||
reg, xregs[reg].guest_reg);
|
reg, xregs[reg].guest_reg);
|
||||||
|
|
||||||
movToMemory(xregs[reg].guest_reg);
|
movToMemory(xregs[reg].guest_reg);
|
||||||
|
|
|
@ -1666,7 +1666,7 @@ static void DoWriteCode(IRBuilder* ibuild, JitIL* Jit, u32 exitAddress) {
|
||||||
u64 val = ibuild->GetImmValue64(I);
|
u64 val = ibuild->GetImmValue64(I);
|
||||||
if ((u32)val == val)
|
if ((u32)val == val)
|
||||||
Jit->MOV(32, R(reg), Imm32(val));
|
Jit->MOV(32, R(reg), Imm32(val));
|
||||||
else if ((s32)val == val)
|
else if ((s32)val == (s64)val)
|
||||||
Jit->MOV(64, R(reg), Imm32(val));
|
Jit->MOV(64, R(reg), Imm32(val));
|
||||||
else
|
else
|
||||||
Jit->MOV(64, R(reg), Imm64(val));
|
Jit->MOV(64, R(reg), Imm64(val));
|
||||||
|
|
|
@ -110,8 +110,6 @@ bool Host_RendererHasFocus()
|
||||||
|
|
||||||
void Host_ConnectWiimote(int wm_idx, bool connect) {}
|
void Host_ConnectWiimote(int wm_idx, bool connect) {}
|
||||||
|
|
||||||
void Host_SetWaitCursor(bool enable){}
|
|
||||||
|
|
||||||
void Host_UpdateStatusBar(const std::string& text, int filed){}
|
void Host_UpdateStatusBar(const std::string& text, int filed){}
|
||||||
|
|
||||||
void Host_SysMessage(const char *fmt, ...)
|
void Host_SysMessage(const char *fmt, ...)
|
||||||
|
|
|
@ -41,15 +41,15 @@
|
||||||
#import <Cocoa/Cocoa.h>
|
#import <Cocoa/Cocoa.h>
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
bool rendererHasFocus = true;
|
static bool rendererHasFocus = true;
|
||||||
bool running = true;
|
static bool running = true;
|
||||||
|
|
||||||
void Host_NotifyMapLoaded() {}
|
void Host_NotifyMapLoaded() {}
|
||||||
void Host_RefreshDSPDebuggerWindow() {}
|
void Host_RefreshDSPDebuggerWindow() {}
|
||||||
|
|
||||||
void Host_ShowJitResults(unsigned int address){}
|
void Host_ShowJitResults(unsigned int address){}
|
||||||
|
|
||||||
Common::Event updateMainFrameEvent;
|
static Common::Event updateMainFrameEvent;
|
||||||
void Host_Message(int Id)
|
void Host_Message(int Id)
|
||||||
{
|
{
|
||||||
switch (Id)
|
switch (Id)
|
||||||
|
@ -109,8 +109,6 @@ bool Host_RendererHasFocus()
|
||||||
|
|
||||||
void Host_ConnectWiimote(int wm_idx, bool connect) {}
|
void Host_ConnectWiimote(int wm_idx, bool connect) {}
|
||||||
|
|
||||||
void Host_SetWaitCursor(bool enable){}
|
|
||||||
|
|
||||||
void Host_UpdateStatusBar(const std::string& text, int filed){}
|
void Host_UpdateStatusBar(const std::string& text, int filed){}
|
||||||
|
|
||||||
void Host_SysMessage(const char *fmt, ...)
|
void Host_SysMessage(const char *fmt, ...)
|
||||||
|
@ -137,7 +135,7 @@ void Host_SetWiiMoteConnectionState(int _State) {}
|
||||||
void Host_ShowVideoConfig(void*, const std::string&, const std::string&) {}
|
void Host_ShowVideoConfig(void*, const std::string&, const std::string&) {}
|
||||||
|
|
||||||
#if HAVE_X11
|
#if HAVE_X11
|
||||||
void X11_MainLoop()
|
static void X11_MainLoop()
|
||||||
{
|
{
|
||||||
bool fullscreen = SConfig::GetInstance().m_LocalCoreStartupParameter.bFullscreen;
|
bool fullscreen = SConfig::GetInstance().m_LocalCoreStartupParameter.bFullscreen;
|
||||||
while (!Core::IsRunning())
|
while (!Core::IsRunning())
|
||||||
|
@ -271,7 +269,7 @@ void X11_MainLoop()
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#if HAVE_WAYLAND
|
#if HAVE_WAYLAND
|
||||||
void Wayland_MainLoop()
|
static void Wayland_MainLoop()
|
||||||
{
|
{
|
||||||
// Wait for display to be initialized
|
// Wait for display to be initialized
|
||||||
while (!GLWin.wl_display)
|
while (!GLWin.wl_display)
|
||||||
|
|
|
@ -21,7 +21,7 @@ void DSPHost::UpdateDebugger() {}
|
||||||
|
|
||||||
// This test goes from text ASM to binary to text ASM and once again back to binary.
|
// This test goes from text ASM to binary to text ASM and once again back to binary.
|
||||||
// Then the two binaries are compared.
|
// Then the two binaries are compared.
|
||||||
bool RoundTrip(const std::vector<u16> &code1)
|
static bool RoundTrip(const std::vector<u16> &code1)
|
||||||
{
|
{
|
||||||
std::vector<u16> code2;
|
std::vector<u16> code2;
|
||||||
std::string text;
|
std::string text;
|
||||||
|
@ -45,7 +45,7 @@ bool RoundTrip(const std::vector<u16> &code1)
|
||||||
|
|
||||||
// This test goes from text ASM to binary to text ASM and once again back to binary.
|
// This test goes from text ASM to binary to text ASM and once again back to binary.
|
||||||
// Very convenient for testing. Then the two binaries are compared.
|
// Very convenient for testing. Then the two binaries are compared.
|
||||||
bool SuperTrip(const char *asm_code)
|
static bool SuperTrip(const char *asm_code)
|
||||||
{
|
{
|
||||||
std::vector<u16> code1, code2;
|
std::vector<u16> code1, code2;
|
||||||
std::string text;
|
std::string text;
|
||||||
|
@ -80,7 +80,7 @@ bool SuperTrip(const char *asm_code)
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
void RunAsmTests()
|
static void RunAsmTests()
|
||||||
{
|
{
|
||||||
bool fail = false;
|
bool fail = false;
|
||||||
#define CHK(a) if (!SuperTrip(a)) printf("FAIL\n%s\n", a), fail = true;
|
#define CHK(a) if (!SuperTrip(a)) printf("FAIL\n%s\n", a), fail = true;
|
||||||
|
|
|
@ -24,7 +24,7 @@ union TestUnion
|
||||||
};
|
};
|
||||||
|
|
||||||
// table of raw numbers to test with
|
// table of raw numbers to test with
|
||||||
u64 table[] =
|
static u64 table[] =
|
||||||
{
|
{
|
||||||
0x0000000000000000ull, // all zero
|
0x0000000000000000ull, // all zero
|
||||||
0xffffffffffffffffull, // all one
|
0xffffffffffffffffull, // all one
|
||||||
|
|
Loading…
Reference in New Issue