From a06642a3a3a8a9c82d6c636cd3f2c18c4fac66c4 Mon Sep 17 00:00:00 2001 From: Argyrios Kyrtzidis Date: Wed, 4 Apr 2012 02:57:01 +0000 Subject: [PATCH] [preprocessor] In Preprocessor::CachingLex() check whether there were more tokens cached during the non-cached lex, otherwise we are going to drop them. Fixes a bogus "_Pragma takes a parenthesized string literal" error when expanding consecutive _Pragmas in a macro argument. Part of rdar://11168596 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@153994 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Lex/PPCaching.cpp | 18 +++++++++++------- test/Preprocessor/_Pragma-in-macro-arg.c | 2 +- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/lib/Lex/PPCaching.cpp b/lib/Lex/PPCaching.cpp index 0d5e34f0bf..6f4c189b4f 100644 --- a/lib/Lex/PPCaching.cpp +++ b/lib/Lex/PPCaching.cpp @@ -57,17 +57,21 @@ void Preprocessor::CachingLex(Token &Result) { ExitCachingLexMode(); Lex(Result); - if (!isBacktrackEnabled()) { + if (isBacktrackEnabled()) { + // Cache the lexed token. + EnterCachingLexMode(); + CachedTokens.push_back(Result); + ++CachedLexPos; + return; + } + + if (CachedLexPos < CachedTokens.size()) { + EnterCachingLexMode(); + } else { // All cached tokens were consumed. CachedTokens.clear(); CachedLexPos = 0; - return; } - - // Cache the lexed token. - EnterCachingLexMode(); - CachedTokens.push_back(Result); - ++CachedLexPos; } void Preprocessor::EnterCachingLexMode() { diff --git a/test/Preprocessor/_Pragma-in-macro-arg.c b/test/Preprocessor/_Pragma-in-macro-arg.c index 99a33a9e1b..2877bcb7bf 100644 --- a/test/Preprocessor/_Pragma-in-macro-arg.c +++ b/test/Preprocessor/_Pragma-in-macro-arg.c @@ -15,7 +15,7 @@ B(_Pragma("clang diagnostic ignored \"-Wparentheses\"")) // This should be ignored.. INACTIVE(_Pragma("clang diagnostic ignored \"-Wconversion\"")) -#define IGNORE_CONV _Pragma("clang diagnostic ignored \"-Wconversion\"") +#define IGNORE_CONV _Pragma("clang diagnostic ignored \"-Wconversion\"") _Pragma("clang diagnostic ignored \"-Wconversion\"") // ..as should this. INACTIVE(IGNORE_CONV) -- 2.50.1