llvm/llvm-project

[Clang] Suggest more test cases on issue 79748

Open

#102.422 aperta il 8 ago 2024

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)C++ (10.782 fork)batch import
clang:frontendgood first issuetest-suite

Metriche repository

Star
 (26.378 star)
Metriche merge PR
 (Merge medio 1g 2h) (1000 PR mergiate in 30 g)

Descrizione

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...

Guida contributor

Direzione di ricerca
Esamina il file di test esistente per operator new in clang, probabilmente situato in clang/test/SemaCXX/ o simile. Esamina il commit 323660ae742aecb524898f956590d599e8869f7f per vedere i casi di test attuali. Aggiungi casi di test per operator new con qualificatori di riferimento (& e &&) come mostrato nell'issue. Usa lo stesso pattern di righe CHECK e messaggi di errore dei test esistenti.
Tech stack
cpp
Dominio
tooling
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80