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

Blr and block map fixes #11840

Merged
merged 2 commits into from
May 23, 2023
Merged

Blr and block map fixes #11840

merged 2 commits into from
May 23, 2023

Conversation

krnlyng
Copy link
Contributor

@krnlyng krnlyng commented May 23, 2023

BLR optimization was completely disabled for both x86 and arm64.

Block map fallback is my bad, sorry.

JosJuice
JosJuice previously approved these changes May 23, 2023
@JosJuice
Copy link
Member

JosJuice commented May 23, 2023

Sorry, my press of the approve button was a little too hasty there. Instead of calling InitBLROptimization in RefreshConfig, could you call it in Jit64::Init and JitArm64::Init, where the code was moved from in b6256a5? I don't believe running InitBLROptimization in the middle of emulation will work as intended, so we shouldn't run it whenever the config is refreshed.

@JosJuice JosJuice dismissed their stale review May 23, 2023 20:47

Missed a potential problem

@AdmiralCurtiss
Copy link
Contributor

What happens if the BLR optimization is valid on boot but then later the user changes some setting that makes it invalid?

@JosJuice
Copy link
Member

Then the BLR optimization continues to be applied for the time being. Fastmem works the same way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants