-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Remove src/test/bench #31274
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Remove src/test/bench #31274
Conversation
Oh man am I jealous of that diffstat 🌴 |
☔ The latest upstream changes (presumably #30900) made this pull request unmergeable. Please resolve the merge conflicts. |
r=me, but needs rebase. |
@bors r=nikomatsakis |
📌 Commit 005c962 has been approved by |
I don't believe these test cases have served any purpose in years. The shootout benchmarks are now upstreamed. A new benchmark suite should rather be maintained out of tree.
⌛ Testing commit 005c962 with merge 578690e... |
⛄ The build was interrupted to prioritize another pull request. |
⌛ Testing commit 005c962 with merge 5c18840... |
💔 Test failed - auto-mac-64-nopt-t |
@bors: retry On Fri, Jan 29, 2016 at 7:53 PM, bors notifications@github.com wrote:
|
⌛ Testing commit 005c962 with merge c4bdc09... |
💔 Test failed - auto-linux-cross-opt |
@bors: retry On Fri, Jan 29, 2016 at 8:53 PM, bors notifications@github.com wrote:
|
I don't believe these test cases have served any purpose in years. The shootout benchmarks are now upstreamed. A new benchmark suite should rather be maintained out of tree. r? @nikomatsakis
…emoved See rust-lang/rust#31274 for the pull request that removed them.
…emoved (#36) See rust-lang/rust#31274 for the pull request that removed them.
I don't believe these test cases have served any purpose in years.
The shootout benchmarks are now upstreamed. A new benchmark suite
should rather be maintained out of tree.
r? @nikomatsakis