Skip to content

Conversation

hkratz
Copy link
Contributor

@hkratz hkratz commented Oct 24, 2021

The rustc fork of Rayon has deadlock detection code which intermittently causes overflows in the CI (see #90227). So, as a workaround, we unconditionally turn overflow checks off for this crate only.

This workaround should be removed once #90227 is fixed.

r? @Mark-Simulacrum

cc @matthiaskrgr

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 24, 2021
@Mark-Simulacrum
Copy link
Member

@bors r+ p=1

@bors
Copy link
Collaborator

bors commented Oct 24, 2021

📌 Commit aacb497 has been approved by Mark-Simulacrum

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 24, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 24, 2021
…askrgr

Rollup of 6 pull requests

Successful merges:

 - rust-lang#89558 (Add rustc lint, warning when iterating over hashmaps)
 - rust-lang#90100 (Skip documentation for tier 2 targets on dist-x86_64-apple-darwin)
 - rust-lang#90155 (Fix alignment of method headings for scannability)
 - rust-lang#90162 (Mark `{array, slice}::{from_ref, from_mut}` as const fn)
 - rust-lang#90221 (Fix ICE when forgetting to `Box` a parameter to a `Self::func` call)
 - rust-lang#90234 (Temporarily turn overflow checks off for rustc-rayon-core)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit eee29fd into rust-lang:master Oct 24, 2021
@bors
Copy link
Collaborator

bors commented Oct 24, 2021

⌛ Testing commit aacb497 with merge 00d5e42...

@rustbot rustbot added this to the 1.58.0 milestone Oct 24, 2021
@hkratz hkratz deleted the rustc-rayon-core-no-overflow-checks branch November 6, 2021 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants