Skip to content

Conversation

bend-n
Copy link
Contributor

@bend-n bend-n commented May 8, 2025

i forgot a few cases rust-lang/rust-clippy#14703

adds

  • char -> i32
  • i32 -> char
  • float -> size ()
  • size -> float
  • i32 -> float
    @rustbot label L-unnecessary_transmutes

@rustbot
Copy link
Collaborator

rustbot commented May 8, 2025

r? @jieyouxu

rustbot has assigned @jieyouxu.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels May 8, 2025
@rustbot
Copy link
Collaborator

rustbot commented May 8, 2025

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

@rustbot rustbot added the L-unnecessary_transmutes Lint: unnecessary_transmutes label May 8, 2025
@rust-log-analyzer

This comment has been minimized.

@jieyouxu
Copy link
Member

jieyouxu commented May 8, 2025

r? @lcnr (as you reviewed #136083)

@rustbot rustbot assigned lcnr and unassigned jieyouxu May 8, 2025
@rustbot
Copy link
Collaborator

rustbot commented May 8, 2025

The Miri subtree was changed

cc @rust-lang/miri

@@ -1,6 +1,7 @@
// We're testing x86 target specific features
//@only-target: x86_64 i686

#![allow(unnecessary_transmutes)]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please leave an empty line between module-level attributes and the module contents.

@rust-log-analyzer

This comment has been minimized.

Copy link
Contributor

@lcnr lcnr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me after the exiting nit by Ralf

@lcnr
Copy link
Contributor

lcnr commented May 9, 2025

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented May 9, 2025

📌 Commit 3b4c493 has been approved by lcnr

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 May 9, 2025
jhpratt added a commit to jhpratt/rust that referenced this pull request May 9, 2025
add signed ints to unn- transmutes to ensure feature parity

i forgot a few cases rust-lang/rust-clippy#14703

adds

- char -> i32
-  i32 -> char
- float -> size ()
-  size -> float
-   i32 -> float
`@rustbot` label L-unnecessary_transmutes
bors added a commit to rust-lang-ci/rust that referenced this pull request May 9, 2025
Rollup of 5 pull requests

Successful merges:

 - rust-lang#140801 (Use span before macro expansion in lint for-loops-over-falibles)
 - rust-lang#140804 (add signed ints to unn- transmutes to ensure feature parity)
 - rust-lang#140812 (Fix `tests/rustdoc-json` triagebot message path)
 - rust-lang#140817 (bootstrap: more consistent use of `...` when citing configuration snippets)
 - rust-lang#140828 (Enable non-leaf Frame Pointers for Arm64 Windows)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request May 9, 2025
Rollup of 5 pull requests

Successful merges:

 - rust-lang#140801 (Use span before macro expansion in lint for-loops-over-falibles)
 - rust-lang#140804 (add signed ints to unn- transmutes to ensure feature parity)
 - rust-lang#140812 (Fix `tests/rustdoc-json` triagebot message path)
 - rust-lang#140817 (bootstrap: more consistent use of `...` when citing configuration snippets)
 - rust-lang#140828 (Enable non-leaf Frame Pointers for Arm64 Windows)

r? `@ghost`
`@rustbot` modify labels: rollup

---
try-job: test-various
bors added a commit to rust-lang-ci/rust that referenced this pull request May 9, 2025
Rollup of 5 pull requests

Successful merges:

 - rust-lang#140801 (Use span before macro expansion in lint for-loops-over-falibles)
 - rust-lang#140804 (add signed ints to unn- transmutes to ensure feature parity)
 - rust-lang#140812 (Fix `tests/rustdoc-json` triagebot message path)
 - rust-lang#140817 (bootstrap: more consistent use of `...` when citing configuration snippets)
 - rust-lang#140828 (Enable non-leaf Frame Pointers for Arm64 Windows)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 16ca18d into rust-lang:master May 9, 2025
6 checks passed
@rustbot rustbot added this to the 1.88.0 milestone May 9, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request May 9, 2025
Rollup merge of rust-lang#140804 - bend-n:signed, r=lcnr

add signed ints to unn- transmutes to ensure feature parity

i forgot a few cases rust-lang/rust-clippy#14703

adds

- char -> i32
-  i32 -> char
- float -> size ()
-  size -> float
-   i32 -> float
``@rustbot`` label L-unnecessary_transmutes
@bend-n bend-n deleted the signed branch May 9, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L-unnecessary_transmutes Lint: unnecessary_transmutes S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants