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

compiletest: Remove jit test support #25754

Merged
merged 1 commit into from
Jul 5, 2015

Conversation

geofft
Copy link
Contributor

@geofft geofft commented May 24, 2015

jit support was removed from rustc in 6b34ba2
(December 2013), so passing --jit wouldn't even work.

@rust-highfive
Copy link
Contributor

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

@bors: r+ fe6b30d

Thanks!

@alexcrichton alexcrichton assigned alexcrichton and unassigned brson May 26, 2015
@bors
Copy link
Collaborator

bors commented May 27, 2015

⌛ Testing commit fe6b30d with merge 18d1f13...

bors added a commit that referenced this pull request May 27, 2015
jit support was removed from rustc in 6b34ba2
(December 2013), so passing `--jit` wouldn't even work.
@bors
Copy link
Collaborator

bors commented May 27, 2015

💔 Test failed - auto-mac-64-opt

@bors
Copy link
Collaborator

bors commented Jun 11, 2015

☔ The latest upstream changes (presumably #26190) made this pull request unmergeable. Please resolve the merge conflicts.

We're no longer passing extra args to the compiler, so remove the variable.
@geofft geofft force-pushed the remove-compiletest-jit branch from fe6b30d to aadc2af Compare June 19, 2015 21:12
@geofft
Copy link
Contributor Author

geofft commented Jun 19, 2015

Oops, @tamird got to this in #26278 while I wasn't paying attention. I've pushed the delta from my branch (eliminating the extra_args parameter that's now unused) but I literally don't care if that change gets merged. We can just close this as a duplicate.

@alexcrichton
Copy link
Member

@bors: r+ aadc2af

Gah sorry this fell through!

bors added a commit that referenced this pull request Jul 5, 2015
jit support was removed from rustc in 6b34ba2
(December 2013), so passing `--jit` wouldn't even work.
@bors
Copy link
Collaborator

bors commented Jul 5, 2015

⌛ Testing commit aadc2af with merge 7c1e339...

@bors bors merged commit aadc2af into rust-lang:master Jul 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants