-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
Open
Labels
C-optimizationCategory: An issue highlighting optimization opportunities or PRs implementing suchCategory: An issue highlighting optimization opportunities or PRs implementing suchT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.needs-triageThis issue may need triage. Remove it if it has been sufficiently triaged.This issue may need triage. Remove it if it has been sufficiently triaged.
Description
This issue was discovered by @ruriww
I tried this code:
#[unsafe(no_mangle)]
pub fn f(x: &mut u32) {
*x = *x;
if *x != 0 {
*x = 0;
}
}
#[unsafe(no_mangle)]
pub fn g(x: &mut u32) {
*x = *x;
if *x != 0 {
*x = 0;
} else {
*x += 0;
}
}I expected both functions to compile to the same assembly. Note that the *x = *x; gives the compiler license to write to *x (without running into opsem questions about whether spurious writes are allowed).
Instead I got the following with -Copt-level=3 (Godbolt):
f:
mov dword ptr [rdi], 0
ret
g:
cmp dword ptr [rdi], 0
je .LBB1_2
mov dword ptr [rdi], 0
.LBB1_2:
retMeta
Godbolt compiler version:
rustc 1.94.0-nightly (fa5eda19b 2025-12-12)
binary: rustc
commit-hash: fa5eda19b95201468f5b1c5c035ec2fc06fccd66
commit-date: 2025-12-12
host: x86_64-unknown-linux-gnu
release: 1.94.0-nightly
LLVM version: 21.1.5
Internal compiler ID: nightly
Metadata
Metadata
Assignees
Labels
C-optimizationCategory: An issue highlighting optimization opportunities or PRs implementing suchCategory: An issue highlighting optimization opportunities or PRs implementing suchT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.needs-triageThis issue may need triage. Remove it if it has been sufficiently triaged.This issue may need triage. Remove it if it has been sufficiently triaged.