[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