[X86][MS]Adjacent comments within multi-line inline assembly statement

[coby]

Summary

Allowing adjacent comments within MS inline assembly multi-line statement

Differential Revision: https://reviews.llvm.org/D28989

[coby]

Ping

[myatsina]

LGTM

[rnk]

Sorry, missed this, looks good.

cfe/trunk/lib/Parse/ParseStmtAsm.cpp

if (!InAsmComment && Tok.is(tok::l_brace)) {
         InAsmComment = false;
         if (isAsm) {
           // If this is a new __asm {} block we want to process it seperately
           // from the single-line __asm statements
           if (PP.LookAhead(0).is(tok::l_brace))
             break;
           LineNo = SrcMgr.getLineNumber(ExpLoc.first, ExpLoc.second);
           SkippedStartOfLine = Tok.isAtStartOfLine();
+        } else if (Tok.is(tok::semi)) {
+          // A multi-line asm-statement, where next line is a comment
+          InAsmComment = true;
+          FID = ExpLoc.first;
+          LineNo = SrcMgr.getLineNumber(FID, ExpLoc.second);
         }
       } else if (!InAsmComment && Tok.is(tok::r_brace)) {
         // In MSVC mode, braces only participate in brace matching and
         // separating the asm statements.  This is an intentional
         // departure from the Apple gcc behavior.
         if (!BraceNesting)
           break;
       }

cfe/trunk/test/CodeGen/ms-inline-asm.c

__asm {
     int 0x2cU ; } asm comments are fun! }{
   }
   __asm {
     {
       int 0x2c ; } asm comments are fun! }{
     }
   }
   __asm {}
+  __asm {
+    ;
+    ; label
+    mov eax, ebx
+  }
 // CHECK: t7
 // CHECK: call void asm sideeffect inteldialect "int $$0x2cU", "~{dirflag},~{fpsr},~{flags}"()
 // CHECK: call void asm sideeffect inteldialect "", "~{dirflag},~{fpsr},~{flags}"()
+// CHECK: call void asm sideeffect inteldialect "mov eax, ebx", "~{eax},~{dirflag},~{fpsr},~{flags}"()
 }
 
 int t8() {
   __asm int 4 ; } comments for single-line asm
   __asm {}
   __asm { int 5}
   __asm int 6
   __asm int 7