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

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


myatsina updated this revision to Diff 30825.

Repository:
  rL LLVM

http://reviews.llvm.org/D11512

Files:
  lib/Target/X86/AsmParser/X86AsmParser.cpp

Index: lib/Target/X86/AsmParser/X86AsmParser.cpp
===================================================================
--- lib/Target/X86/AsmParser/X86AsmParser.cpp
+++ lib/Target/X86/AsmParser/X86AsmParser.cpp
@@ -912,6 +912,11 @@
   if (RegNo == 0)
     RegNo = MatchRegisterName(Tok.getString().lower());
 
+  if (isParsingInlineAsm() && isParsingIntelSyntax() && RegNo == X86::EFLAGS) {
+    // Cannot address reg called "flags" in MS inline asm - this is just an identifier!
+    RegNo = 0;
+  }
+
   if (!is64BitMode()) {
     // FIXME: This should be done using Requires<Not64BitMode> and
     // Requires<In64BitMode> so "eiz" usage in 64-bit instructions can be also


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11512.30825.patch
Type: text/x-patch
Size: 685 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150728/c56c26b9/attachment.bin>


More information about the llvm-commits mailing list