[asan-assembly-instrumentation] Fixed operations with esp for x86-32.

[ygorshenin]

Summary

[asan-assembly-instrumentation] Fixed operations with esp for x86-32.

[eugenis]

LGTM
This is covered by compiler-rt test in D3839.
I don't think a meaningful IR-level test is possible here.

[eugenis]

r209280.

lib/Target/X86/AsmParser/X86AsmInstrumentation.cpp

void X86AddressSanitizer32::InstrumentMemOperandImpl(
   EmitInstruction(Out, MCInstBuilder(X86::PUSH32r).addReg(X86::EAX));
   {
     const std::string Func = FuncName(AccessSize, IsWrite);
     const MCSymbol *FuncSym = Ctx.GetOrCreateSymbol(StringRef(Func));
     const MCSymbolRefExpr *FuncExpr =
         MCSymbolRefExpr::Create(FuncSym, MCSymbolRefExpr::VK_PLT, Ctx);
     EmitInstruction(Out, MCInstBuilder(X86::CALLpcrel32).addExpr(FuncExpr));
   }
-  EmitInstruction(Out, MCInstBuilder(X86::ADD32ri).addReg(X86::ESP)
-                           .addReg(X86::ESP).addImm(4));
+  EmitInstruction(Out, MCInstBuilder(X86::POP32r).addReg(X86::EAX));
   EmitInstruction(Out, MCInstBuilder(X86::POP32r).addReg(X86::EAX));
 }
 
 class X86AddressSanitizer64 : public X86AddressSanitizer {
 public:
   X86AddressSanitizer64(const MCSubtargetInfo &STI)
       : X86AddressSanitizer(STI) {}
   virtual ~X86AddressSanitizer64() {}