Use multiple threads by default. Limits tests to one thread. Do some renaming.#57948
Use multiple threads by default. Limits tests to one thread. Do some renaming.#57948bors merged 3 commits intorust-lang:masterfrom
Conversation
rust-highfive
added
the
S-waiting-on-review
This comment has been minimized.
This comment has been minimized.
| // The interner is pointed to by a thread local value which is only set on the main thread | ||
| // with parallelization is disabled. So we don't allow `Symbol` to transfer between threads | ||
| // to avoid panics and other errors, even though it would be memory safe to do so. | ||
| #[cfg(not(parallel_queries))] |
There was a problem hiding this comment.
CFGs like these seem very prone to invisible breakage (if we end up not running either configuration on CI). Perhaps we should maintain parallel-able code in all configurations?
There was a problem hiding this comment.
We do check both on PRs on Travis at least.
| "print the AST as JSON and halt"), | ||
| query_threads: Option<usize> = (None, parse_opt_uint, [UNTRACKED], | ||
| "execute queries on a thread pool with N threads"), | ||
| threads: Option<usize> = (None, parse_opt_uint, [UNTRACKED], |
There was a problem hiding this comment.
Is there any support for jobserver or is this a static quantity? Perhaps the documentation should be adjusted either way?
There was a problem hiding this comment.
It will use a jobserver once #56946 lands
This comment has been minimized.
This comment has been minimized.
|
(I'm assuming we will want a perf run here?) @bors try |
|
⌛ Trying commit fd9d9ee with merge a8571c0a49643e461ab10f2410709f5a9e9c1b00... |
|
☀️ Test successful - checks-travis |
|
📌 Commit fd9d9ee has been approved by |
bors
added
S-waiting-on-bors
Use multiple threads by default. Limits tests to one thread. Do some renaming. r? @michaelwoerister
|
☀️ Test successful - checks-travis, status-appveyor |
|
☀️ Test successful - checks-travis, status-appveyor |
6 participants
r? @michaelwoerister