Poor codegen for u64::isqrt()
Nessuno ha ancora preso questa issue.
- Lingua principale
- Rust
- Stelle
- 119k
- Fork
- 16.2k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
I tried this code:
fn isqrt(val:u64)->u64{
val.isqrt()
}
I expected to see this happen: rustc generates reasonable code.
Instead, this happened: rustc generates ~150 too many instructions, which takes a while to run.
Potential fix: Use this implementation instead:
fn isqrt(val:u64)->u64{
let estimate = (val as f64).sqrt() as u64;
if val < (1<< 52){//value fits in mantissa, exact precision.
return estimate;
}else{
// Otherwise, do a newton-raphson iteration for the final bits of precision
unsafe{assert_unchecked(estimate!=0);};
return (estimate + val/estimate)/2;
}
}
On my machine, it ran twice as fast.
Meta
rustc --version --verbose:
rustc 1.92.0 (ded5c06cf 2025-12-08)
binary: rustc
commit-hash: ded5c06cf21d2b93bffd5d884aa6e96934ee4234
commit-date: 2025-12-08
host: x86_64-pc-windows-msvc
release: 1.92.0
LLVM version: 21.1.3
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia compilando l’esempio fornito di u64::isqrt() con la toolchain rustc 1.92.0 indicata e analizzando le istruzioni generate. Traccia il percorso di generazione del codice di rustc per questa operazione e confronta il relativo output e tempo di esecuzione con l’approccio proposto nell’issue; il lavoro è completo quando le istruzioni in eccesso e il rallentamento sono stati risolti con evidenze provenienti da un benchmark mirato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- rust
- Ambito
- compilers, performance
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 38/100