Core: Add option to break on address exception
This commit is contained in:
parent
cffeceef70
commit
10d23486c6
|
@ -246,6 +246,7 @@ CJniBridegSettings::CJniBridegSettings()
|
|||
// Debugger
|
||||
ADD_SETTING(Debugger_Enabled);
|
||||
ADD_SETTING(Debugger_BreakOnUnhandledMemory);
|
||||
ADD_SETTING(Debugger_BreakOnAddressError);
|
||||
ADD_SETTING(Debugger_ShowPifErrors);
|
||||
ADD_SETTING(Debugger_ShowDivByZero);
|
||||
ADD_SETTING(Debugger_RecordRecompilerAsm);
|
||||
|
|
|
@ -204,17 +204,9 @@ void CRegisters::CheckInterrupts()
|
|||
|
||||
void CRegisters::DoAddressError(bool DelaySlot, uint32_t BadVaddr, bool FromRead)
|
||||
{
|
||||
if (HaveDebugger())
|
||||
if (BreakOnAddressError())
|
||||
{
|
||||
g_Notify->DisplayError(stdstr_f("AddressError %s Vaddr: %X", FromRead ? "reading" : "writing", BadVaddr).c_str());
|
||||
if ((STATUS_REGISTER & STATUS_EXL) != 0)
|
||||
{
|
||||
g_Notify->DisplayError("EXL set in AddressError exception");
|
||||
}
|
||||
if ((STATUS_REGISTER & STATUS_ERL) != 0)
|
||||
{
|
||||
g_Notify->DisplayError("ERL set in AddressError exception");
|
||||
}
|
||||
g_Notify->BreakPoint(__FILE__, __LINE__);
|
||||
}
|
||||
|
||||
if (FromRead)
|
||||
|
|
|
@ -320,6 +320,7 @@ void CSettings::AddHowToHandleSetting(const char * BaseDirectory)
|
|||
|
||||
AddHandler(Debugger_Enabled, new CSettingTypeApplication("Debugger", "Debugger", false));
|
||||
AddHandler(Debugger_BreakOnUnhandledMemory, new CSettingTypeApplication("Debugger", "Break On Unhandled Memory", false));
|
||||
AddHandler(Debugger_BreakOnAddressError, new CSettingTypeApplication("Debugger", "Break On Address Error", false));
|
||||
AddHandler(Debugger_ShowPifErrors, new CSettingTypeApplication("Debugger", "Show Pif Errors", false));
|
||||
AddHandler(Debugger_DisableGameFixes, new CSettingTypeApplication("Debugger", "Disable Game Fixes", false));
|
||||
AddHandler(Debugger_ShowDListAListCount, new CSettingTypeApplication("Debugger", "Show Dlist Alist Count", false));
|
||||
|
|
|
@ -24,6 +24,7 @@ uint32_t CDebugSettings::m_FpExceptionBreakpoints = 0;
|
|||
uint32_t CDebugSettings::m_IntrBreakpoints = 0;
|
||||
uint32_t CDebugSettings::m_RcpIntrBreakpoints = 0;
|
||||
bool CDebugSettings::m_BreakOnUnhandledMemory = false;
|
||||
bool CDebugSettings::m_BreakOnAddressError = false;
|
||||
|
||||
CDebugSettings::CDebugSettings()
|
||||
{
|
||||
|
@ -48,6 +49,7 @@ CDebugSettings::CDebugSettings()
|
|||
g_Settings->RegisterChangeCB(Debugger_IntrBreakpoints, this, (CSettings::SettingChangedFunc)StaticRefreshSettings);
|
||||
g_Settings->RegisterChangeCB(Debugger_RcpIntrBreakpoints, this, (CSettings::SettingChangedFunc)StaticRefreshSettings);
|
||||
g_Settings->RegisterChangeCB(Debugger_BreakOnUnhandledMemory, this, (CSettings::SettingChangedFunc)StaticRefreshSettings);
|
||||
g_Settings->RegisterChangeCB(Debugger_BreakOnAddressError, this, (CSettings::SettingChangedFunc)StaticRefreshSettings);
|
||||
|
||||
RefreshSettings();
|
||||
}
|
||||
|
@ -74,6 +76,7 @@ CDebugSettings::~CDebugSettings()
|
|||
g_Settings->UnregisterChangeCB(Debugger_IntrBreakpoints, this, (CSettings::SettingChangedFunc)StaticRefreshSettings);
|
||||
g_Settings->UnregisterChangeCB(Debugger_RcpIntrBreakpoints, this, (CSettings::SettingChangedFunc)StaticRefreshSettings);
|
||||
g_Settings->UnregisterChangeCB(Debugger_BreakOnUnhandledMemory, this, (CSettings::SettingChangedFunc)StaticRefreshSettings);
|
||||
g_Settings->UnregisterChangeCB(Debugger_BreakOnAddressError, this, (CSettings::SettingChangedFunc)StaticRefreshSettings);
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -96,6 +99,7 @@ void CDebugSettings::RefreshSettings()
|
|||
m_IntrBreakpoints = m_HaveDebugger ? g_Settings->LoadDword(Debugger_IntrBreakpoints) : 0;
|
||||
m_RcpIntrBreakpoints = m_HaveDebugger ? g_Settings->LoadDword(Debugger_RcpIntrBreakpoints) : 0;
|
||||
m_BreakOnUnhandledMemory = m_HaveDebugger && g_Settings->LoadBool(Debugger_BreakOnUnhandledMemory);
|
||||
m_BreakOnAddressError = m_HaveDebugger && g_Settings->LoadBool(Debugger_BreakOnAddressError);
|
||||
|
||||
m_Debugging = m_HaveDebugger && (m_HaveExecutionBP || m_WaitingForStep || m_HaveWriteBP || m_HaveReadBP);
|
||||
}
|
|
@ -27,6 +27,7 @@ public:
|
|||
static inline uint32_t IntrBreakpoints(void) { return m_IntrBreakpoints; }
|
||||
static inline uint32_t RcpIntrBreakpoints(void) { return m_RcpIntrBreakpoints; }
|
||||
static inline bool BreakOnUnhandledMemory(void) { return m_BreakOnUnhandledMemory; }
|
||||
static inline bool BreakOnAddressError(void) { return m_BreakOnAddressError; }
|
||||
|
||||
private:
|
||||
static void StaticRefreshSettings(CDebugSettings * _this)
|
||||
|
@ -55,6 +56,7 @@ private:
|
|||
static uint32_t m_IntrBreakpoints;
|
||||
static uint32_t m_RcpIntrBreakpoints;
|
||||
static bool m_BreakOnUnhandledMemory;
|
||||
static bool m_BreakOnAddressError;
|
||||
|
||||
static int32_t m_RefCount;
|
||||
static bool m_Registered;
|
||||
|
|
|
@ -237,6 +237,7 @@ enum SettingID
|
|||
// Debugger
|
||||
Debugger_Enabled,
|
||||
Debugger_BreakOnUnhandledMemory,
|
||||
Debugger_BreakOnAddressError,
|
||||
Debugger_ShowPifErrors,
|
||||
Debugger_ShowDivByZero,
|
||||
Debugger_RecordRecompilerAsm,
|
||||
|
|
|
@ -25,6 +25,7 @@ CMainMenu::CMainMenu(CMainGui * hMainWindow) :
|
|||
m_ChangeSettingList.push_back(Logging_GenerateLog);
|
||||
m_ChangeSettingList.push_back(Debugger_RecordExecutionTimes);
|
||||
m_ChangeSettingList.push_back(Debugger_BreakOnUnhandledMemory);
|
||||
m_ChangeSettingList.push_back(Debugger_BreakOnAddressError);
|
||||
m_ChangeSettingList.push_back(Debugger_ShowPifErrors);
|
||||
m_ChangeSettingList.push_back(Debugger_ShowDListAListCount);
|
||||
m_ChangeSettingList.push_back(Debugger_DebugLanguage);
|
||||
|
@ -499,6 +500,9 @@ bool CMainMenu::ProcessMessage(HWND hWnd, DWORD /*FromAccelerator*/, DWORD MenuI
|
|||
case ID_DEBUG_BREAK_ON_UNHANDLED_MEM:
|
||||
g_Settings->SaveBool(Debugger_BreakOnUnhandledMemory, !g_Settings->LoadBool(Debugger_BreakOnUnhandledMemory));
|
||||
break;
|
||||
case ID_DEBUG_BREAK_ON_ADDRESS_ERROR:
|
||||
g_Settings->SaveBool(Debugger_BreakOnAddressError, !g_Settings->LoadBool(Debugger_BreakOnAddressError));
|
||||
break;
|
||||
case ID_DEBUG_SHOW_PIF_ERRORS:
|
||||
g_Settings->SaveBool(Debugger_ShowPifErrors, !g_Settings->LoadBool(Debugger_ShowPifErrors));
|
||||
break;
|
||||
|
@ -1223,6 +1227,12 @@ void CMainMenu::FillOutMenu(HMENU hMenu)
|
|||
Item.SetItemTicked(true);
|
||||
}
|
||||
DebugNotificationMenu.push_back(Item);
|
||||
Item.Reset(ID_DEBUG_BREAK_ON_ADDRESS_ERROR, EMPTY_STRING, EMPTY_STDSTR, nullptr, L"Break on address error");
|
||||
if (g_Settings->LoadBool(Debugger_BreakOnAddressError))
|
||||
{
|
||||
Item.SetItemTicked(true);
|
||||
}
|
||||
DebugNotificationMenu.push_back(Item);
|
||||
Item.Reset(ID_DEBUG_SHOW_PIF_ERRORS, EMPTY_STRING, EMPTY_STDSTR, nullptr, L"On PIF errors");
|
||||
if (g_Settings->LoadBool(Debugger_ShowPifErrors))
|
||||
{
|
||||
|
|
|
@ -33,7 +33,7 @@ enum MainMenuID
|
|||
ID_OPTIONS_DECREASE_SPEED,
|
||||
|
||||
// Debugger menu
|
||||
ID_DEBUG_BREAK_ON_UNHANDLED_MEM, ID_DEBUG_SHOW_PIF_ERRORS,
|
||||
ID_DEBUG_BREAK_ON_UNHANDLED_MEM, ID_DEBUG_BREAK_ON_ADDRESS_ERROR, ID_DEBUG_SHOW_PIF_ERRORS,
|
||||
ID_DEBUG_SHOW_DLIST_COUNT, ID_DEBUG_SHOW_RECOMP_MEM_SIZE, ID_DEBUG_SHOW_DIV_BY_ZERO,
|
||||
ID_DEBUG_RECORD_RECOMPILER_ASM, ID_DEBUG_DISABLE_GAMEFIX, ID_DEBUG_LANGUAGE,
|
||||
ID_DEBUGGER_LOGOPTIONS, ID_DEBUGGER_GENERATELOG, ID_DEBUGGER_DUMPMEMORY, ID_DEBUGGER_SEARCHMEMORY,
|
||||
|
|
Loading…
Reference in New Issue