[PATCH] Add support for nested blocks in Microsoft inline assembly

David Majnemer david.majnemer at gmail.com
Sat Jul 5 22:18:49 PDT 2014


LGTM with requested changes addressed.

================
Comment at: lib/Parse/ParseStmtAsm.cpp:463
@@ -432,3 +462,3 @@
   }
 
   // If we don't support assembly, or the assembly is empty, we don't
----------------
We should probably assert that LBraceLocs isn't empty at this point.

http://reviews.llvm.org/D4399






More information about the cfe-commits mailing list