`blake2` code size very high (overuse of`#[inline(always)]`?)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- rust
- Área
- cryptography
Línea de trabajo
Comienza en blake2/src/blake2.rs, en VarBlake2b::compress, centrándote en las anotaciones #[inline(always)] y en las llamadas a las rondas desenrolladas manualmente. Compara el código generado con -Copt-level=z y -Copt-level=3 y, después, verifica que la implementación siga produciendo los mismos resultados de Blake2b mientras reduce el tamaño inusualmente grande de la función.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Rust
- Estrellas
- 2.3k
- Forks
- 341
- Merge medio
- 12 h 31 min
- PR fusionados (30 d)
- 2
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de RustCrypto/hashes
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
RustCrypto/hashes#912 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
RustCrypto/hashes#909 · 4 comentarios ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
RustCrypto/hashes#908 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
RustCrypto/hashes#763 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
RustCrypto/hashes#747 · 1 comentario ·
Todos los issues de RustCrypto/hashes
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
TheLarkInn/aipm#2413 ·
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
alexgorbatchev/simple-ptt#15 ·
-
tooling
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
todo:ticket
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
taikoxyz/taiko-mono#22168 · 1 comentario ·