llvm/llvm-project

[Clang] Suggest more test cases on issue 79748

Geschlossen

#102.422 geöffnet am 08.08.2024

Auf GitHub ansehenContributor Guide
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (10.782 Forks)batch import
clang:frontendgood first issuetest-suite

Repository-Metriken

Stars
 (26.378 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Sieh dir die vorhandene Testdatei für operator new in clang an, wahrscheinlich in clang/test/SemaCXX/ oder ähnlich. Untersuche den Commit 323660ae742aecb524898f956590d599e8869f7f, um die aktuellen Testfälle zu sehen. Füge Testfälle für operator new mit Referenzqualifizierern (& und &&) hinzu, wie im Issue gezeigt. Verwende dieselben CHECK Zeilen und Fehlermeldungsmuster wie in den bestehenden Tests.
Tech Stack
cpp
Domain
tooling
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80