summaryrefslogtreecommitdiffstats
path: root/clang
diff options
context:
space:
mode:
authorEli Friedman <eli.friedman@gmail.com>2009-05-24 19:25:46 +0000
committerEli Friedman <eli.friedman@gmail.com>2009-05-24 19:25:46 +0000
commit2d49c4f98de60b11ff56a6ae6399162f1aaa2db8 (patch)
treebae7d8b33eedc2993368f544aa5f4a452fe17596 /clang
parent4486da5b78b47dc3c947ceb634d13a3d0ff0889e (diff)
downloadbcm5719-llvm-2d49c4f98de60b11ff56a6ae6399162f1aaa2db8.tar.gz
bcm5719-llvm-2d49c4f98de60b11ff56a6ae6399162f1aaa2db8.zip
Make sure an invalid concatentaion doesn't insert whitespace before
the RHS. Fixes assembler-with-cpp issue reported on cfe-dev. llvm-svn: 72370
Diffstat (limited to 'clang')
-rw-r--r--clang/lib/Lex/TokenLexer.cpp6
-rw-r--r--clang/test/Preprocessor/assembler-with-cpp.c5
2 files changed, 10 insertions, 1 deletions
diff --git a/clang/lib/Lex/TokenLexer.cpp b/clang/lib/Lex/TokenLexer.cpp
index f1af42aea51..65d083dffdc 100644
--- a/clang/lib/Lex/TokenLexer.cpp
+++ b/clang/lib/Lex/TokenLexer.cpp
@@ -223,7 +223,11 @@ void TokenLexer::ExpandFunctionArguments() {
// If the next token was supposed to get leading whitespace, ensure it has
// it now.
if (CurTok.hasLeadingSpace() || NextTokGetsSpace) {
- ResultToks[ResultToks.size()-NumToks].setFlag(Token::LeadingSpace);
+ // Exception: the RHS of a paste doesn't get whitespace. This allows
+ // constructs like conacatenating a period and an identifer to work
+ // correctly in assembler-with-cpp.
+ if (!PasteBefore)
+ ResultToks[ResultToks.size()-NumToks].setFlag(Token::LeadingSpace);
NextTokGetsSpace = false;
}
continue;
diff --git a/clang/test/Preprocessor/assembler-with-cpp.c b/clang/test/Preprocessor/assembler-with-cpp.c
index 885e67b98c8..bb168805785 100644
--- a/clang/test/Preprocessor/assembler-with-cpp.c
+++ b/clang/test/Preprocessor/assembler-with-cpp.c
@@ -63,4 +63,9 @@ T7(foo)
// RUN: grep 'T6 #nostring' %t &&
// RUN: grep 'T7 "foo"' %t &&
+// Concatenation with period doesn't leave a space
+// RUN: grep '.T8' %t &&
+#define T8(A,B) A ## B
+T8(.,T8)
+
// RUN: true
OpenPOWER on IntegriCloud