summaryrefslogtreecommitdiffstats
path: root/clang
diff options
context:
space:
mode:
authorArgyrios Kyrtzidis <akyrtzi@gmail.com>2013-01-29 20:28:24 +0000
committerArgyrios Kyrtzidis <akyrtzi@gmail.com>2013-01-29 20:28:24 +0000
commit005206a81362156299d4e928162b6b669c456918 (patch)
treed0a8a5ccc2b8ace8a1ddc85abf8c5c2acdac604c /clang
parentdd60e0902144a7e9a162a6f9aa6d333075c389a4 (diff)
downloadbcm5719-llvm-005206a81362156299d4e928162b6b669c456918.tar.gz
bcm5719-llvm-005206a81362156299d4e928162b6b669c456918.zip
[Preprocessor] When checking if we can concatenate two tokens, check
if they were already concatenated in source using the spelling locations even if they came from a macro expansion. This fixes an issue where a GUID passed as macro argument ends up malformed after preprocessing because we added spaces inside it. rdar://13016645 llvm-svn: 173826
Diffstat (limited to 'clang')
-rw-r--r--clang/lib/Lex/TokenConcatenation.cpp7
-rw-r--r--clang/test/Preprocessor/output_paste_avoid.c6
2 files changed, 10 insertions, 3 deletions
diff --git a/clang/lib/Lex/TokenConcatenation.cpp b/clang/lib/Lex/TokenConcatenation.cpp
index f7b594594f0..30dc8f19d90 100644
--- a/clang/lib/Lex/TokenConcatenation.cpp
+++ b/clang/lib/Lex/TokenConcatenation.cpp
@@ -156,9 +156,10 @@ bool TokenConcatenation::AvoidConcat(const Token &PrevPrevTok,
// First, check to see if the tokens were directly adjacent in the original
// source. If they were, it must be okay to stick them together: if there
// were an issue, the tokens would have been lexed differently.
- if (PrevTok.getLocation().isFileID() && Tok.getLocation().isFileID() &&
- PrevTok.getLocation().getLocWithOffset(PrevTok.getLength()) ==
- Tok.getLocation())
+ SourceManager &SM = PP.getSourceManager();
+ SourceLocation PrevSpellLoc = SM.getSpellingLoc(PrevTok.getLocation());
+ SourceLocation SpellLoc = SM.getSpellingLoc(Tok.getLocation());
+ if (PrevSpellLoc.getLocWithOffset(PrevTok.getLength()) == SpellLoc)
return false;
tok::TokenKind PrevKind = PrevTok.getKind();
diff --git a/clang/test/Preprocessor/output_paste_avoid.c b/clang/test/Preprocessor/output_paste_avoid.c
index 8e4f3a4bf4d..6defd1272b7 100644
--- a/clang/test/Preprocessor/output_paste_avoid.c
+++ b/clang/test/Preprocessor/output_paste_avoid.c
@@ -31,3 +31,9 @@ E: test(str)
#define equal =
F: >>equal
// CHECK: F: >> =
+
+// Make sure we don't introduce spaces in the guid because we try to avoid
+// pasting '-' to a numeric constant.
+#define TYPEDEF(guid) typedef [uuid(guid)]
+TYPEDEF(66504301-BE0F-101A-8BBB-00AA00300CAB) long OLE_COLOR;
+// CHECK: typedef [uuid(66504301-BE0F-101A-8BBB-00AA00300CAB)] long OLE_COLOR;
OpenPOWER on IntegriCloud