[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

[Chad Rosier]

On Jan 10, 2013, at 5:16 PM, NAKAMURA Takumi <geek4civic at gmail.com> wrote:

> 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.

Thanks, Nakamura!