WebAssembly / WebAssembly/binaryen
Change default inlining behavior to favor single-caller inlines
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- WebAssembly
- Estrellas
- 8.6k
- Forks
- 885
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 69
Descripción
See FIXMEs introduced in: https://github.com/WebAssembly/binaryen/pull/2125
Essentially, the check for calls == 1 need to be moved first in that function, otherwise oneCallerInlineMaxSize can never be bigger than flexibleInlineMaxSize, which naturally it would make sense for it to be.
@kripken mentioned this change can only be made after it has been checked if it doesn't regress size/speed.
Guía de contribución
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.
Línea de trabajo
Comienza con los cambios FIXME introducidos por el pull request 2125 y localiza la función de inlining que contiene la comprobación calls == 1. Verifica el cambio de orden mediante mediciones de tamaño y velocidad antes de considerar terminado el trabajo; el issue no proporciona ningún archivo específico de prueba o benchmark.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp, wasm
- Área
- compilers, performance
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100