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

Move two macros (JITDISABLE and INSTRUCTION_START) into JitBase.h. #18

Merged
merged 1 commit into from Feb 1, 2014

Conversation

lioncash
Copy link
Member

Reduces 'copypasta' of macros across JIT platforms.

Keeps it nice and centralized on one macro set.

…s (JITDISABLE and INSTRUCTION_START) and simply have it in the JitBase.h source file. This way the macro is just located in one spot rather than 6 places.
@lioncash
Copy link
Member Author

@Sonicadvance1 Can I get your input on this?

@ghost ghost assigned Sonicadvance1 Jan 31, 2014
@degasus
Copy link
Member

degasus commented Jan 31, 2014

LGTM

@BhaaLseN
Copy link
Member

+1 (LGTM? Lets Get it To Master?)

@delroth
Copy link
Member

delroth commented Jan 31, 2014

Looks Good To Me.

On Fri, Jan 31, 2014 at 8:10 PM, BhaaL notifications@github.com wrote:

+1 (LGTM? Lets Get it To Master?)

Reply to this email directly or view it on GitHubhttps://github.com//pull/18#issuecomment-33832336
.

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

@Sonicadvance1
Copy link
Contributor

LGTM

delroth added a commit that referenced this pull request Feb 1, 2014
Move two macros (JITDISABLE and INSTRUCTION_START) into JitBase.h.
@delroth delroth merged commit 97119d0 into dolphin-emu:master Feb 1, 2014
@lioncash lioncash deleted the small-jit-macro-cleanup branch February 1, 2014 17:32
CarlKenner referenced this pull request in CarlKenner/dolphin Dec 19, 2014
Another difficult merge done.
@ghost ghost mentioned this pull request Feb 11, 2021
rapito pushed a commit to rapito/dolphin that referenced this pull request Sep 16, 2022
…res-implementation

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