// From the MS website: If used without braces, the __asm keyword means
// that the rest of the line is an assembly-language statement.
SourceManager &SrcMgr = PP.getSourceManager();
- unsigned lineNo = SrcMgr.getLineNumber(Tok.getLocation());
- do {
- ConsumeAnyToken();
- } while ((SrcMgr.getLineNumber(Tok.getLocation()) == lineNo) &&
- Tok.isNot(tok::r_brace) && Tok.isNot(tok::eof));
+ SourceLocation TokLoc = Tok.getLocation();
+ if (TokLoc.isFileID()) {
+ unsigned lineNo = SrcMgr.getLineNumber(TokLoc);
+ do {
+ ConsumeAnyToken();
+ TokLoc = Tok.getLocation();
+ } while (TokLoc.isFileID() && (SrcMgr.getLineNumber(TokLoc) == lineNo) &&
+ Tok.isNot(tok::r_brace) && Tok.isNot(tok::semi) &&
+ Tok.isNot(tok::eof));
+ } else { // The asm tokens come from a macro expansion.
+ do {
+ ConsumeAnyToken();
+ TokLoc = Tok.getLocation();
+ } while (TokLoc.isMacroID() && Tok.isNot(tok::r_brace) &&
+ Tok.isNot(tok::semi) && Tok.isNot(tok::eof));
+ }
}
return false;
}
--- /dev/null
+// RUN: clang %s -verify -fms-extensions
+
+#define M __asm int 0x2c
+#define M2 int
+
+void t1(void) { M }
+void t2(void) { __asm int 0x2c }
+// FIXME? We don't support fuzzy parsing line-oriented __asm's where the body is partially defined in a macro.
+void t3(void) { __asm M2 0x2c } // expected-error{{expected ';' after expression}} expected-warning{{expression result unused}}
+