Rollup of 5 pull requests#157902
Closed
jhpratt wants to merge 11 commits into
Closed
Conversation
* update intrinsics wrapping documentation * fix variable names
* stabilize feature float_algebraic * fixed algrebracis for `f16` and `f128`
Update LLVM to 22.1.7 Update for rust-lang/llvm-project#196. LLVM 22.1.7 contains fixes for the PowerPC specific issues produced by Rust.
…loat_algebraic, r=RalfJung stabilize feature `float_algebraic` Closes rust-lang#136469 Thanks to @RalfJung for getting the stabilization process started and asking me to draft the stabilization PR for the `float_algebraic` feature. This will be my first contribution to rust! I've done my best to follow the various guides for stabilization PRs, but this is my first one, so do give it a careful look. The only place left that includes "float_algebraic" is line 8202 of "src/tools/rust-analyzer/crates/ide-db/src/generated/lints.rs", but I'm assuming that's fine.
…-op-type, r=camelid `suspicious_double_ref_op`: report unadjusted return type `suspicious_double_ref_op` uses the adjusted expression type to distinguish a no-op call from a double-reference operation. When the result is used as another method's receiver, that type can include the parent method's receiver adjustment. For example, `file.clone().write(&[])` reports that `clone` returns `&mut &File`, even though it returns `&File`. This PR continues to use the adjusted type for lint classification, but now uses the method call's unadjusted type in the diagnostic. The same issue affects `.deref()`, so the regression test covers both operations. Fixes rust-lang#146227
…jhpratt Stabilize `nonzero_from_str_radix` Closes rust-lang#152193
Update intrinsics wrapping documentation
This PR updates the documentation for `intrinsics::{wrapping_add, wrapping_sub, wrapping_mul}`,
to make it accurate for both signed and unsigned types.
Related issue: rust-lang#156726
rustbot
added
A-LLVM
Member
Author
|
@bors r+ rollup=never p=5 |
Contributor
rust-bors
Bot
added
S-waiting-on-bors
rust-bors
Bot
added
S-waiting-on-author
Contributor
|
This pull request was unapproved due to being closed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Successful merges:
float_algebraic#157029 (stabilize featurefloat_algebraic)suspicious_double_ref_op: report unadjusted return type #157872 (suspicious_double_ref_op: report unadjusted return type)nonzero_from_str_radix#157877 (Stabilizenonzero_from_str_radix)r? @ghost
Create a similar rollup