diff --git a/lib/Lex/TokenLexer.cpp b/lib/Lex/TokenLexer.cpp index 8c6f6204eb838e9826a9d3377dd5979259d528ae..5b41fe9b8d3f2a77def2edb5993cb0b4bef95dcd 100644 --- a/lib/Lex/TokenLexer.cpp +++ b/lib/Lex/TokenLexer.cpp @@ -749,14 +749,18 @@ static void updateConsecutiveMacroArgTokens(SourceManager &SM, Token *NextTok = begin_tokens + 1; for (; NextTok < end_tokens; ++NextTok) { + SourceLocation NextLoc = NextTok->getLocation(); + if (CurLoc.isFileID() != NextLoc.isFileID()) + break; // Token from different kind of FileID. + int RelOffs; - if (!SM.isInSameSLocAddrSpace(CurLoc, NextTok->getLocation(), &RelOffs)) + if (!SM.isInSameSLocAddrSpace(CurLoc, NextLoc, &RelOffs)) break; // Token from different local/loaded location. // Check that token is not before the previous token or more than 50 // "characters" away. if (RelOffs < 0 || RelOffs > 50) break; - CurLoc = NextTok->getLocation(); + CurLoc = NextLoc; } // For the consecutive tokens, find the length of the SLocEntry to contain diff --git a/test/Preprocessor/macro_arg_slocentry_merge.c b/test/Preprocessor/macro_arg_slocentry_merge.c new file mode 100644 index 0000000000000000000000000000000000000000..9ab385f80841f3486625cc611157abd74001c8de --- /dev/null +++ b/test/Preprocessor/macro_arg_slocentry_merge.c @@ -0,0 +1,7 @@ +// RUN: not %clang_cc1 -fsyntax-only %s 2>&1 | FileCheck %s + +#include "macro_arg_slocentry_merge.h" + +// CHECK: macro_arg_slocentry_merge.h:7:19: error: unknown type name 'win' +// CHECK: macro_arg_slocentry_merge.h:5:16: note: expanded from macro 'WINDOW' +// CHECK: macro_arg_slocentry_merge.h:6:18: note: expanded from macro 'P_' diff --git a/test/Preprocessor/macro_arg_slocentry_merge.h b/test/Preprocessor/macro_arg_slocentry_merge.h new file mode 100644 index 0000000000000000000000000000000000000000..62595b762e5ecf16279733caf0fa12854bfac6b5 --- /dev/null +++ b/test/Preprocessor/macro_arg_slocentry_merge.h @@ -0,0 +1,7 @@ + + + + +#define WINDOW win +#define P_(args) args +extern void f P_((WINDOW win));