llvm/llvm-project

[Attributor] Make use of analysis in the MustBeExecutedExplorer

Aberta

#175 aberto em 13 de mar. de 2020

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)C++ (10.782 forks)batch import
good first issuegsochelp wantedllvm::Attributor

Métricas do repositório

Stars
 (26.378 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Right now the MustBeExecutedContextExplorer Explorer is created without analysis, e.g., without a dominator tree. As a consequence it is not able to determine non-trivial control flow merge points which reduces the "must-be-executed-context" substantially.

The task is to provide the appropriate analysis getters to the Explorer during creation and update the tests accordingly. If tests for the new capabilities are missing, thus the affected tests do not show the expected effect, new tests need to be added.

Guia do colaborador

Direção de pesquisa
Entenda a classe MustBeExecutedContextExplorer e identifique quais getters de análise são necessários (por exemplo, árvore de dominância). Em seguida, atualize os testes existentes e adicione novos testes para verificar o comportamento melhorado.
Pilha de tecnologia
cpp
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.
5
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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.
Principalmente claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50