llvm/llvm-project

[clang-tidy] llvm-header-guard suggest invalid ifdef macros name

Open

#166.750 aperta il 6 nov 2025

Vedi su GitHub
 (10 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
clang-tidyfalse-positivegood first issue

Metriche repository

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

Descrizione

When https://clang.llvm.org/extra/clang-tidy/checks/llvm/header-guard.html check sees a file with has "llvm" inside its filepath (e.g. clang-tools-extra/clang-tidy/llvm/HeaderGuardCheck.h), instead of making such ifdef:

#ifndef LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_LLVM_HEADERGUARDCHECK_H

The check makes:

#ifndef LLVM_HEADERGUARDCHECK_H

The check removes leading part if sees "LLVM" in check-name which is incorrect.

Guida contributor

Direzione di ricerca
Individua il codice di generazione dell'header guard in clang tidy LLVMHeaderGuardCheck e correggi la logica che tronca erroneamente il percorso quando 'LLVM' appare nel percorso del file.
Tech stack
cpp
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60