New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Jit: Move most x86-64-specific code out of JitCommon #4516
Conversation
4b4175d
to
c786067
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice cleanup.
@@ -18,25 +18,8 @@ | |||
#include "Core/PowerPC/CPUCoreBase.h" | |||
#include "Core/PowerPC/Jit64Common/Jit64AsmCommon.h" |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
static const int FARCODE_SIZE = 1024 * 1024 * 8; | ||
static const int FARCODE_SIZE_MMU = 1024 * 1024 * 48; | ||
constexpr int FARCODE_SIZE = 1024 * 1024 * 8; | ||
constexpr int FARCODE_SIZE_MMU = 1024 * 1024 * 48; |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
@@ -21,7 +21,6 @@ | |||
#include "Common/x64Emitter.h" | |||
#include "Core/PowerPC/Gekko.h" | |||
#include "Core/PowerPC/Jit64/JitAsm.h" |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
Mostly intended to clean up HandleFault than anything else.
LGTM |
Moves it over to Jit64Common, as well as very minor cleanup that follows the primary commit.