Skip to content
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

JitArm64: Avoid loading compilerPC multiple times if it's already in a register. #12086

Merged
merged 1 commit into from Aug 27, 2023

Conversation

krnlyng
Copy link
Contributor

@krnlyng krnlyng commented Aug 3, 2023

Makes the generated code slightly less redundant in some cases.

@krnlyng krnlyng force-pushed the jitcompilerPCload branch 3 times, most recently from c013175 to c2beb84 Compare August 4, 2023 14:56
Source/Core/Core/PowerPC/JitArm64/Jit.cpp Outdated Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/Jit.cpp Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/Jit.cpp Outdated Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/Jit.cpp Outdated Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/JitArm64_Branch.cpp Outdated Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/JitArm64_Branch.cpp Outdated Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/JitArm64_Branch.cpp Outdated Show resolved Hide resolved
@krnlyng krnlyng force-pushed the jitcompilerPCload branch 2 times, most recently from 5527bea to c7a304e Compare August 26, 2023 12:39
@krnlyng krnlyng force-pushed the jitcompilerPCload branch 2 times, most recently from cf24abf to e90d18b Compare August 26, 2023 13:27
Source/Core/Core/PowerPC/JitArm64/Jit.cpp Outdated Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/Jit.cpp Outdated Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/Jit.cpp Outdated Show resolved Hide resolved
Source/Core/Core/PowerPC/JitArm64/JitArm64_Branch.cpp Outdated Show resolved Hide resolved
@krnlyng krnlyng force-pushed the jitcompilerPCload branch 2 times, most recently from 18bb29b to 60af7d3 Compare August 26, 2023 13:43
@JosJuice
Copy link
Member

Ah, seems like my merge of a different PR just now caused a merge conflict...

@krnlyng
Copy link
Contributor Author

krnlyng commented Aug 26, 2023

Ah, seems like my merge of a different PR just now caused a merge conflict...

Yep looking into it.

@JosJuice JosJuice merged commit 7ac0db7 into dolphin-emu:master Aug 27, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants