swiftlang/swift

[SR-6264] Unify duplicated code in GlobalPropertyOpt::isVisibleExternally() and FunctionLivenessComputation::isVisibleExternally()

Aberta

#48.814 aberto em 1 de nov. de 2017

Ver no GitHubGuia do colaborador
 (4 comentários) (0 reação) (0 responsável)Swift (10.719 forks)batch import
IRGencompilergood first issueimprovement

Métricas do repositório

Stars
 (69.989 estrelas)
Métricas de merge de PR
 (Mesclagem média 8d 17h) (510 fundiu PRs em 30d)

Description
Previous ID SR-6264
Radar None
Original Reporter @gparker42
Type Improvement
Votes 0
Component/s Compiler
Labels Improvement, IRGen, StarterBug
Assignee None
Priority Medium

md5: 7fe2948a0ff34a9d22f375125e2fe476

Issue Description:

Functions GlobalPropertyOpt::isVisibleExternally() and FunctionLivenessComputation::isVisibleExternally() have duplicated code that checks whether a declaration may be externally used based on its access level. This duplication should be fixed.

Possible solution: add a function to class ValueDecl to do the work, and call it from both places.

Guia do colaborador

Direção de pesquisa
Examine as duas funções GlobalPropertyOpt::isVisibleExternally() e FunctionLivenessComputation::isVisibleExternally() no código fonte do compilador Swift. É provável que estejam em arquivos relacionados ao IRGen e SILOptimizer. Identifique a lógica duplicada e crie um novo método na classe ValueDecl que encapsule a verificação do nível de acesso. Em seguida, substitua o código duplicado em ambas as funções por chamadas a esse novo método. Certifique se de que os testes continuem passando.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Refatoração
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.
Recente
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.
80