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

Jit64: Fix immediates being zero-extended to cr_val instead of sign-extended. #727

Merged
merged 1 commit into from Aug 3, 2014

Conversation

delroth
Copy link
Member

@delroth delroth commented Aug 3, 2014

Thanks to konpie for finding this issue. See https://code.google.com/p/dolphin-emu/issues/detail?id=7538 .

…xtended. Thanks to konpie for finding this issue.
@Tilka
Copy link
Member

Tilka commented Aug 3, 2014

lgtm

1 similar comment
@phire
Copy link
Member

phire commented Aug 3, 2014

lgtm

@Sonicadvance1
Copy link
Contributor

LOOOOOOOOOOOOOOOKS GOOOOOOOOOOOOOOOOOOOD TOOOOOOOOOOOOOOOOO MEEEEEEEEEEEEEEEEE~

Sonicadvance1 added a commit that referenced this pull request Aug 3, 2014
Jit64: Fix immediates being zero-extended to cr_val instead of sign-extended.
@Sonicadvance1 Sonicadvance1 merged commit ee7b861 into dolphin-emu:master Aug 3, 2014
@delroth
Copy link
Member Author

delroth commented Aug 3, 2014

@JMC47 could you do some basic regression testing on this PR?

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