emscripten-core/emscripten

Add option to disable warnings when ASYNCIFY_REMOVE functions are not found

Open

#13.179 aberto em 3 de jan. de 2021

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (3.519 forks)batch import
asyncifyhelp wanted

Métricas do repositório

Stars
 (27.361 stars)
Métricas de merge de PR
 (Mesclagem média 19d 10h) (147 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Encontre onde o aviso sobre a função ASYNCIFY REMOVE não encontrada é emitido no código fonte do Emscripten (provavelmente na integração binaryen). Adicione uma nova opção de linha de comando como ' asyncify remove warnings' para suprimir esse aviso. Modifique o código relevante para verificar essa opção antes de imprimir o aviso.
Pilha de tecnologia
None
Domain
compilers
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70