WebAssembly / WebAssembly/binaryen
Change default inlining behavior to favor single-caller inlines
Personne n'a encore pris cette issue.
- Langage dominant
- WebAssembly
- Étoiles
- 8.6k
- Forks
- 885
- Merge moyen
- 1 j 19 h
- PR mergées (30 j)
- 69
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par les changements FIXME introduits par la pull request 2125 et localisez la fonction d'inlining contenant la vérification calls == 1. Vérifiez le changement d'ordre à l'aide de mesures de taille et de vitesse avant de considérer le travail comme terminé ; l'issue ne fournit aucun fichier de test ou de benchmark spécifique.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- cpp, wasm
- Domaine
- compilers, performance
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100