-
Notifications
You must be signed in to change notification settings - Fork 119
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
Fix loading roms larger than 64MB #499
base: develop
Are you sure you want to change the base?
Conversation
I do have a hard time believing this is all thats required |
Go ahead and try it out. This one romhack called b3313 surpasses 64MB on 0.9 and later versions, and this build fixes it. |
I dont need to try it, at best this will allow data access and maybe interpreter but i am positive code execution from that will fail |
mupen64plus/mupen64plus-core#997 |
gee I wonder who it could be, if only there was something common in the url & the guy who made the issue or something edit: in hindsight I could have made the above joke ^ in a less passive aggressive manner, apologies for that... |
You can run code from uncached cart dom |
Well it is super slow because only uncached works and i think it only works in interpreter rn anyway.. Either way this PR should be aimed upstream I guess, I might allow it if they decline tho I could add support for dynarec however.. Probably only some homebrew test roms even use that feature tho, data access is most common |
This one character fix makes the core able to load roms that are larger than 64MB in size. For example Super Mario 64 romhacks made using the older tools can sometimes surpass 64MB, and mupen64plus_next isn't able to run them correctly without this fix.