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