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

Device: RDRAM: Use the R4300 helpers to get the R4300 registers #503

Merged
merged 1 commit into from Dec 18, 2017

Conversation

Projects
None yet
2 participants
@Miouyouyou
Copy link
Contributor

commented Dec 17, 2017

Since the "regs" member is not available on the r4300_core
structure when enabling ARM DYNAREC, accessing this member
directly will cause compilation errors with this setup.

Signed-off-by: Myy Miouyouyou myy@miouyouyou.fr

Device: RDRAM: Use the R4300 helpers to get the R4300 registers
Since the "regs" member is not available on the r4300_core
structure when enabling ARM DYNAREC, accessing this member
directly will cause compilation errors with this setup.

Signed-off-by: Myy Miouyouyou <myy@miouyouyou.fr>
@bsmiles32

This comment has been minimized.

Copy link
Member

commented Dec 17, 2017

Thanks for your PR. It looks good and was indeed an oversight from my part.
Unfortunately our AppVeyor instance is stuck at the moment so CI checks will take a while before completing. But after, I'll merge that.

Good Work !

@bsmiles32 bsmiles32 merged commit f5ea0d1 into mupen64plus:master Dec 18, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.