diff options
| author | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2013-01-30 20:42:17 +0000 |
|---|---|---|
| committer | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2013-01-30 20:42:17 +0000 |
| commit | a94bd438f8ddc3013b5122d2ba1648ce5a428df0 (patch) | |
| tree | 047abd96efcb1901e2797e0398cbdfefdcf5b992 | |
| parent | 1330eccb5f8efd0bf6b272d8241326a3a4780e30 (diff) | |
| download | bcm5719-llvm-a94bd438f8ddc3013b5122d2ba1648ce5a428df0.tar.gz bcm5719-llvm-a94bd438f8ddc3013b5122d2ba1648ce5a428df0.zip | |
Revert "[preprocessor] Don't warn about "disabled expansion of recursive macro""
This reverts commit r173952
llvm-svn: 173970
| -rw-r--r-- | clang/lib/Lex/PPMacroExpansion.cpp | 5 | ||||
| -rw-r--r-- | clang/test/Headers/stdbool.cpp | 5 | ||||
| -rw-r--r-- | clang/test/Preprocessor/warn-disabled-macro-expansion.c | 2 |
3 files changed, 2 insertions, 10 deletions
diff --git a/clang/lib/Lex/PPMacroExpansion.cpp b/clang/lib/Lex/PPMacroExpansion.cpp index 939c4faf7a0..63791448858 100644 --- a/clang/lib/Lex/PPMacroExpansion.cpp +++ b/clang/lib/Lex/PPMacroExpansion.cpp @@ -459,10 +459,7 @@ bool Preprocessor::HandleMacroExpandedIdentifier(Token &Identifier, if (MacroInfo *NewMI = getMacroInfo(NewII)) if (!NewMI->isEnabled() || NewMI == MI) { Identifier.setFlag(Token::DisableExpand); - // Don't warn for "#define X X" like "#define bool bool" from - // stdbool.h. - if (NewMI != MI) - Diag(Identifier, diag::pp_disabled_macro_expansion); + Diag(Identifier, diag::pp_disabled_macro_expansion); } } diff --git a/clang/test/Headers/stdbool.cpp b/clang/test/Headers/stdbool.cpp index 7c927db441b..a252cca1a6a 100644 --- a/clang/test/Headers/stdbool.cpp +++ b/clang/test/Headers/stdbool.cpp @@ -1,6 +1,5 @@ // RUN: %clang_cc1 -E -dM %s | FileCheck --check-prefix=CHECK-GNU-COMPAT %s // RUN: %clang_cc1 -std=c++98 -E -dM %s | FileCheck --check-prefix=CHECK-CONFORMING %s -// RUN: %clang_cc1 -fsyntax-only -std=gnu++98 -verify -Weverything %s #include <stdbool.h> #define zzz @@ -13,7 +12,3 @@ // CHECK-CONFORMING: #define __CHAR_BIT__ // CHECK-CONFORMING-NOT: #define false false // CHECK-CONFORMING: #define zzz - -zzz -// expected-no-diagnostics -extern bool x; diff --git a/clang/test/Preprocessor/warn-disabled-macro-expansion.c b/clang/test/Preprocessor/warn-disabled-macro-expansion.c index c97a4ee4431..b01b63f6b3f 100644 --- a/clang/test/Preprocessor/warn-disabled-macro-expansion.c +++ b/clang/test/Preprocessor/warn-disabled-macro-expansion.c @@ -16,7 +16,7 @@ #define z(x) (z)(x) -p // no warning +p // expected-warning {{recursive macro}} a // expected-warning {{recursive macro}} |
