Skip to content

Conversation

Skgland
Copy link
Contributor

@Skgland Skgland commented Apr 7, 2025

I discovered #132735 (comment) that I might have done too much in #132735 by also removing support in r-a.
So this reverts the commit with the changes to r-a.

r? RalfJung

@rustbot
Copy link
Collaborator

rustbot commented Apr 7, 2025

rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead.

cc @rust-lang/rust-analyzer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 7, 2025
@Veykril
Copy link
Member

Veykril commented Apr 7, 2025

That depends on the last Rust version that was actually making use of rust-intrinsic ABI in the standard library. Iirc our MSRV for project support is 1.78 or so right now

@Veykril
Copy link
Member

Veykril commented Apr 7, 2025

Ah I see #136005 removed the last bits, yes then we need to revert this again.

@Veykril
Copy link
Member

Veykril commented Apr 7, 2025

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 7, 2025

📌 Commit bc29123 has been approved by Veykril

It is now in the queue for this repository.

@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 Apr 7, 2025
@Skgland
Copy link
Contributor Author

Skgland commented Apr 7, 2025

Iirc our MSRV for project support is 1.78 or so right now

Looks to be correct based on https://github.com/rust-lang/rust-analyzer/blob/c5882732e6e6e09ac75cddd13545e95860be1c42/crates/rust-analyzer/src/lib.rs#L13-L19

@RalfJung
Copy link
Member

RalfJung commented Apr 7, 2025

r? @Veykril (to reflect reality)

@rustbot rustbot assigned Veykril and unassigned RalfJung Apr 7, 2025
Zalathar added a commit to Zalathar/rust that referenced this pull request Apr 8, 2025
…kril

Revert r-a changes of rust-lang#139455

I discovered rust-lang#132735 (comment) that I might have done too much in rust-lang#132735 by also removing support in r-a.
So this reverts the commit with the changes to r-a.

r? RalfJung
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 8, 2025
…errors

Rollup of 19 pull requests

Successful merges:

 - rust-lang#138676 (Implement overflow for infinite implied lifetime bounds)
 - rust-lang#139024 (Make error message for missing fields with `..` and without `..` more consistent)
 - rust-lang#139098 (Tell LLVM about impossible niche tags)
 - rust-lang#139124 (compiler: report error when trait object type param reference self)
 - rust-lang#139321 (Update to new rinja version (askama))
 - rust-lang#139346 (Don't construct preds w escaping bound vars in `diagnostic_hir_wf_check`)
 - rust-lang#139386 (make it possible to use stage0 libtest on compiletest)
 - rust-lang#139421 (Fix trait upcasting to dyn type with no principal when there are projections)
 - rust-lang#139468 (Don't call `Span::with_parent` on the good path in `has_stashed_diagnostic`)
 - rust-lang#139476 (rm `RegionInferenceContext::var_infos`)
 - rust-lang#139481 (Add job summary links to post-merge report)
 - rust-lang#139485 (compiletest: Stricter parsing for diagnostic kinds)
 - rust-lang#139490 (Update some comment/docs related to "extern intrinsic" removal)
 - rust-lang#139491 (Update books)
 - rust-lang#139496 (Revert r-a changes of rust-lang#139455)
 - rust-lang#139500 (document panic behavior of Vec::resize and Vec::resize_with)
 - rust-lang#139501 (Fix stack overflow in exhaustiveness due to recursive HIR opaque hidden types)
 - rust-lang#139504 (add missing word in doc comment)
 - rust-lang#139507 (compiletest: Trim whitespace from environment variable names)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 8, 2025
…iaskrgr

Rollup of 10 pull requests

Successful merges:

 - rust-lang#139494 (Restrict some queries by def-kind more)
 - rust-lang#139496 (Revert r-a changes of rust-lang#139455)
 - rust-lang#139506 (add missing word in doc comment (part 2))
 - rust-lang#139515 (Improve presentation of closure signature mismatch from `Fn` trait goal)
 - rust-lang#139520 (compiletest maintenance: sort deps and drop dep on `anyhow`)
 - rust-lang#139523 (Rustc dev guide subtree update)
 - rust-lang#139526 (Fix deprecation note for std::intrinsics)
 - rust-lang#139528 (compiletest: Remove the `--logfile` flag)
 - rust-lang#139541 (Instantiate higher-ranked transmute goal w/ placeholders before emitting sub-obligations)
 - rust-lang#139547 (Update library tracking issue template to set S-tracking-unimplemented)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 9, 2025
…iaskrgr

Rollup of 10 pull requests

Successful merges:

 - rust-lang#139494 (Restrict some queries by def-kind more)
 - rust-lang#139496 (Revert r-a changes of rust-lang#139455)
 - rust-lang#139506 (add missing word in doc comment (part 2))
 - rust-lang#139515 (Improve presentation of closure signature mismatch from `Fn` trait goal)
 - rust-lang#139520 (compiletest maintenance: sort deps and drop dep on `anyhow`)
 - rust-lang#139523 (Rustc dev guide subtree update)
 - rust-lang#139526 (Fix deprecation note for std::intrinsics)
 - rust-lang#139528 (compiletest: Remove the `--logfile` flag)
 - rust-lang#139541 (Instantiate higher-ranked transmute goal w/ placeholders before emitting sub-obligations)
 - rust-lang#139547 (Update library tracking issue template to set S-tracking-unimplemented)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 7661224 into rust-lang:master Apr 9, 2025
6 checks passed
@rustbot rustbot added this to the 1.88.0 milestone Apr 9, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Apr 9, 2025
Rollup merge of rust-lang#139496 - Skgland:139455-went-too-far, r=Veykril

Revert r-a changes of rust-lang#139455

I discovered rust-lang#132735 (comment) that I might have done too much in rust-lang#132735 by also removing support in r-a.
So this reverts the commit with the changes to r-a.

r? RalfJung
@Skgland Skgland deleted the 139455-went-too-far branch April 9, 2025 08:56
github-actions bot pushed a commit to model-checking/verify-rust-std that referenced this pull request Apr 19, 2025
…iaskrgr

Rollup of 10 pull requests

Successful merges:

 - rust-lang#139494 (Restrict some queries by def-kind more)
 - rust-lang#139496 (Revert r-a changes of rust-lang#139455)
 - rust-lang#139506 (add missing word in doc comment (part 2))
 - rust-lang#139515 (Improve presentation of closure signature mismatch from `Fn` trait goal)
 - rust-lang#139520 (compiletest maintenance: sort deps and drop dep on `anyhow`)
 - rust-lang#139523 (Rustc dev guide subtree update)
 - rust-lang#139526 (Fix deprecation note for std::intrinsics)
 - rust-lang#139528 (compiletest: Remove the `--logfile` flag)
 - rust-lang#139541 (Instantiate higher-ranked transmute goal w/ placeholders before emitting sub-obligations)
 - rust-lang#139547 (Update library tracking issue template to set S-tracking-unimplemented)

r? `@ghost`
`@rustbot` modify labels: rollup
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