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

x64Emitter: add support for shorter EAX forms of instructions #941

Merged
merged 1 commit into from Sep 3, 2014

Conversation

FioraAeterna
Copy link
Contributor

Should save a few bytes of code size here and there.

@comex
Copy link
Contributor

comex commented Sep 2, 2014

Mostly unrelated, but nops is a somewhat confusing variable name for a list of normal ops.

@delroth
Copy link
Member

delroth commented Sep 2, 2014

Agreed. Any way you could change that while you're at it?

On Tue, Sep 2, 2014 at 5:46 PM, comex notifications@github.com wrote:

Mostly unrelated, but nops is a somewhat confusing variable name for a
list of normal ops.


Reply to this email directly or view it on GitHub
#941 (comment).

Pierre "delroth" Bourdon delroth@gmail.com
Software Engineer @ Zürich, Switzerland
http://code.delroth.net/

Should save a few bytes of code size here and there.
shuffle2 added a commit that referenced this pull request Sep 3, 2014
x64Emitter: add support for shorter EAX forms of instructions
@shuffle2 shuffle2 merged commit 5468b53 into dolphin-emu:master Sep 3, 2014
Sintendo added a commit to Sintendo/dolphin that referenced this pull request Jan 12, 2020
TODO was already taken care of in PR dolphin-emu#941.
Sintendo added a commit to Sintendo/dolphin that referenced this pull request Jan 13, 2020
TODO was already taken care of in PR dolphin-emu#941.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants