[PATCH] D11512: Fix conflict for referencing "flags" variable in X86 assembly syntax

Marina Yatsina marina.yatsina at intel.com
Tue Jul 28 09:21:05 PDT 2015


myatsina added a comment.

I've added "isParsingIntelSyntax" because we should not encounter this issue in att syntax (for att memory location has a $ sign - $flags)


================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:915
@@ -914,1 +914,3 @@
 
+  if (isParsingInlineAsm() && isParsingIntelSyntax() && RegNo == X86::EFLAGS) {
+    // Cannot address reg called "flags" in MS inline asm - this is just an identifier!
----------------
I've added "isParsingIntelSyntax" because we should not encounter this issue in att syntax (for att memory location has a $ sign - $flags)


Repository:
  rL LLVM

http://reviews.llvm.org/D11512







More information about the llvm-commits mailing list