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

Remove JITIL #5445

Merged
merged 1 commit into from May 20, 2017

Conversation

5 participants
@JosJuice
Contributor

JosJuice commented May 19, 2017

  • It's unmaintained
  • There's essentially no reason for users to use it instead of the regular x86-64 JIT
  • It's preventing us from making PIE builds (PR #5271)
  • Probably some other reasons too
@Helios747

I am 100% in favor of this. I don't like having 63783 options that confuse users. And Cached Interpreter is more useful than this anyways.

@waddlesplash

This comment has been minimized.

Show comment
Hide comment
@waddlesplash

waddlesplash May 20, 2017

Contributor

Can I get credit for supporting this, even if I was 3 years too early (#1023)? 😛

Contributor

waddlesplash commented May 20, 2017

Can I get credit for supporting this, even if I was 3 years too early (#1023)? 😛

@lioncash

This comment has been minimized.

Show comment
Hide comment
@lioncash

lioncash May 20, 2017

Member

Reppin' PIE and ASLR city~

Member

lioncash commented May 20, 2017

Reppin' PIE and ASLR city~

@lioncash lioncash merged commit b47d44a into dolphin-emu:master May 20, 2017

9 of 10 checks passed

pr-android Build failed on builder pr-android
Details
default Very basic checks passed, handed off to Buildbot.
Details
lint Build succeeded on builder lint
Details
pr-deb-dbg-x64 Build succeeded on builder pr-deb-dbg-x64
Details
pr-deb-x64 Build succeeded on builder pr-deb-x64
Details
pr-freebsd-x64 Build succeeded on builder pr-freebsd-x64
Details
pr-osx-x64 Build succeeded on builder pr-osx-x64
Details
pr-ubu-x64 Build succeeded on builder pr-ubu-x64
Details
pr-win-dbg-x64 Build succeeded on builder pr-win-dbg-x64
Details
pr-win-x64 Build succeeded on builder pr-win-x64
Details
@Kolano

This comment has been minimized.

Show comment
Hide comment

Kolano commented May 20, 2017

@JosJuice JosJuice deleted the JosJuice:remove-jitil branch May 20, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment