[llvm-commits] [llvm] r172121 - in /llvm/trunk: include/llvm/MC/MCParser/MCAsmParser.h include/llvm/MC/MCParser/MCParsedAsmOperand.h lib/MC/MCParser/AsmParser.cpp lib/Target/X86/AsmParser/X86AsmParser.cpp test/CodeGen/X86/ms-inline-asm.ll

NAKAMURA Takumi geek4civic at gmail.com
Thu Jan 10 17:16:17 PST 2013


2013/1/11 Chad Rosier <mcrosier at apple.com>:
> Thanks, Nakamura.  I believe r172148 should have taken care of the problem as well.

You should've added one more;

--- a/llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp
+++ b/llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp
@@ -528,6 +528,7 @@ struct X86Operand : public MCParsedAsmOperand {
     Res->Mem.Scale    = Scale;
     Res->Mem.Size     = Size;
     Res->Mem.NeedSizeDir = NeedSizeDir;
+    Res->AddressOf = false;
     return Res;
   }
 };

Applied in r172157. The tree would be fine even if my r172153 were reverted.



More information about the llvm-commits mailing list