`blake2` code size very high (overuse of`#[inline(always)]`?)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- rust
- Ambito
- cryptography
Direzione di ricerca
Inizia in blake2/src/blake2.rs, in VarBlake2b::compress, concentrandoti sulle annotazioni #[inline(always)] e sulle chiamate ai round srotolati manualmente. Confronta il codice generato con -Copt-level=z e -Copt-level=3, quindi verifica che l'implementazione continui a produrre gli stessi risultati Blake2b riducendo al contempo le dimensioni insolitamente grandi della funzione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I have ~150kb of code for blake2::blake2b::VarBlake2b::compress under -Copt-level=z, which is around 3x larger than any other function in my binary. It would be nice if it were smaller.
Looking at https://github.com/RustCrypto/hashes/blob/master/blake2/src/blake2.rs, a lot is #[inline(always)], and the round calls are manually unrolled.
There's a good chance these would still get inlined and unrolled on -Copt-level=3 if it were written using #[inline] and a loop, but it would make a big difference to use cases where large code is undesirable.
- Lingua principale
- Rust
- Stelle
- 2.3k
- Fork
- 341
- Merge medio
- 12h 31m
- PR unite (30g)
- 2
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
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.
Altre issue di RustCrypto/hashes
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
RustCrypto/hashes#912 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
RustCrypto/hashes#909 · 4 commenti ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
RustCrypto/hashes#908 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
RustCrypto/hashes#763 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
RustCrypto/hashes#747 · 1 commento ·
Tutte le issue di RustCrypto/hashes
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
TheLarkInn/aipm#2413 ·
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
alexgorbatchev/simple-ptt#15 ·
-
tooling
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
todo:ticket
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
taikoxyz/taiko-mono#22168 · 1 commento ·