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

Remove more checks for LLVM < 12 #90623

Merged
merged 4 commits into from
Nov 6, 2021
Merged

Remove more checks for LLVM < 12 #90623

merged 4 commits into from
Nov 6, 2021

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Nov 5, 2021

We already updated the minimum to 12 in #90175, but we missed a few get_version() checks.

@rust-highfive
Copy link
Collaborator

r? @jackh726

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 5, 2021
@jackh726
Copy link
Member

jackh726 commented Nov 5, 2021

r? @nagisa

@rust-highfive rust-highfive assigned nagisa and unassigned jackh726 Nov 5, 2021
@cuviper
Copy link
Member Author

cuviper commented Nov 5, 2021

There are a few more comments around NoAliasMutRef -- let me get those too...

@nikic
Copy link
Contributor

nikic commented Nov 5, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Nov 5, 2021

📌 Commit 767471e has been approved by nikic

@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 Nov 5, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 5, 2021
…askrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#89942 (Reorder `widening_impl`s to make the doc clearer)
 - rust-lang#90569 (Fix tests using `only-i686` to use the correct `only-x86` directive)
 - rust-lang#90597 (Warn for variables that are no longer captured)
 - rust-lang#90623 (Remove more checks for LLVM < 12)
 - rust-lang#90626 (Properly register text_direction_codepoint_in_comment lint.)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 9032b9d into rust-lang:master Nov 6, 2021
@rustbot rustbot added this to the 1.58.0 milestone Nov 6, 2021
@cuviper cuviper deleted the llvm-12 branch December 8, 2021 07:29
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.

7 participants