From a33e0504f61132ac43f4f89af43b12c87e987c20 Mon Sep 17 00:00:00 2001 From: Argyrios Kyrtzidis Date: Tue, 18 Jan 2011 19:50:15 +0000 Subject: [PATCH] When redefining a macro don't warn twice if it's not used and don't warn for duplicate definition by command line options. Fixes rdar://8875916. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@123767 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Lex/PPDirectives.cpp | 4 +++- test/Preprocessor/warn-macro-unused.c | 5 +++++ 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 test/Preprocessor/warn-macro-unused.c diff --git a/lib/Lex/PPDirectives.cpp b/lib/Lex/PPDirectives.cpp index f422d254f0..0f0d25b887 100644 --- a/lib/Lex/PPDirectives.cpp +++ b/lib/Lex/PPDirectives.cpp @@ -1527,7 +1527,7 @@ void Preprocessor::HandleDefineDirective(Token &DefineTok) { // then don't bother calling MacroInfo::isIdenticalTo. if (!getDiagnostics().getSuppressSystemWarnings() || !SourceMgr.isInSystemHeader(DefineTok.getLocation())) { - if (!OtherMI->isUsed()) + if (!OtherMI->isUsed() && OtherMI->isWarnIfUnused()) Diag(OtherMI->getDefinitionLoc(), diag::pp_macro_not_used); // Macros must be identical. This means all tokens and whitespace @@ -1539,6 +1539,8 @@ void Preprocessor::HandleDefineDirective(Token &DefineTok) { Diag(OtherMI->getDefinitionLoc(), diag::note_previous_definition); } } + if (OtherMI->isWarnIfUnused()) + WarnUnusedMacroLocs.erase(OtherMI->getDefinitionLoc()); ReleaseMacroInfo(OtherMI); } diff --git a/test/Preprocessor/warn-macro-unused.c b/test/Preprocessor/warn-macro-unused.c new file mode 100644 index 0000000000..8a6d7c25ca --- /dev/null +++ b/test/Preprocessor/warn-macro-unused.c @@ -0,0 +1,5 @@ +// RUN: %clang_cc1 %s -Wunused-macros -Dfoo -Dfoo -verify + +#define unused // expected-warning {{macro is not used}} +#define unused +unused -- 2.50.1