emscripten-core/emscripten

Add option to disable warnings when ASYNCIFY_REMOVE functions are not found

Ouverte

#13 179 ouverte le 3 janv. 2021

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)C++ (3 519 forks)batch import
asyncifyhelp wanted

Métriques du dépôt

Stars
 (27 361 étoiles)
Métriques de merge PR
 (Merge moyen 19j 10h) (147 PRs mergées en 30 j)

Description

My project compiles a number of related apps which make slightly different calls into an ASYNCIFY using library. I've determined there are a number of functions which are safe to include in ASYNCIFY_REMOVE, but not all of the apps call all of the functions. It is a little frustrating to have to manually track for each app which functions are called and which aren't, in order to avoid the warnings Binaryen prints.

It would be nice if there was an option to disable the Binaryen warnings when an ASYNCIFY_REMOVE function is not found.

Guide contributeur

Direction de recherche
Trouvez l'endroit où l'avertissement concernant la fonction ASYNCIFY REMOVE introuvable est émis dans le code source d'Emscripten (probablement dans l'intégration binaryen). Ajoutez une nouvelle option de ligne de commande comme ' asyncify remove warnings' pour supprimer cet avertissement. Modifiez le code concerné pour vérifier cette option avant d'afficher l'avertissement.
Stack technique
Aucun
Domaine
compilers
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70