[PATCH] D17767: [ms-inline-asm][AVX512] Add ability to use k registers in MS inline asm + fix bag with curly braces

Marina Yatsina via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 7 00:05:43 PST 2016


myatsina added inline comments.

================
Comment at: lib/MC/MCParser/AsmParser.cpp:4989
@@ +4988,3 @@
+
+    if (Lexer.is(AsmToken::LCurly) || Lexer.is(AsmToken::RCurly)) {
+      // Identify curly brace marking block start/end
----------------
delena wrote:
> Can this code be moved into parseStatement()?
If we this code to parseStatement() then .s files that contain blocks with braces will be legal.
I don't think we want llvm-mc to be able to accept such syntax.


Repository:
  rL LLVM

http://reviews.llvm.org/D17767





More information about the llvm-commits mailing list