llvm/llvm-project

[Clang] Suggest more test cases on issue 79748

Fechada

#102.422 aberto em 8 de ago. de 2024

Ver no GitHubGuia do colaborador
 (7 comentários) (0 reação) (0 responsável)C++ (10.782 forks)batch import
clang:frontendgood first issuetest-suite

Métricas do repositório

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

Description

This is just a robustness enhancement suggestion.

https://github.com/llvm/llvm-project/issues/79748

I triggered this issue and found the solution done in https://github.com/llvm/llvm-project/issues/79748. However, the test cases attached to https://github.com/agozillon/llvm-project/commit/323660ae742aecb524898f956590d599e8869f7f do not include those for reference qualifiers, in addition to cv-qualifiers. As for the completeness consideration, I suggest to add more test cases like:

void* operator new(size_t bytes) &;  //expected-error...
void* operator new(size_t bytes) &&;  //expected-error...

Guia do colaborador

Direção de pesquisa
Examine o arquivo de teste existente para operator new no clang, provavelmente localizado em clang/test/SemaCXX/ ou similar. Analise o commit 323660ae742aecb524898f956590d599e8869f7f para ver os casos de teste atuais. Adicione casos de teste para operator new com qualificadores de referência (& e &&) conforme mostrado na issue. Use as mesmas linhas CHECK e padrão de mensagens de erro dos testes existentes.
Pilha de tecnologia
cpp
Domain
tooling
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80