[PATCH] Added address sanitizer instrumentation for MOV and MOVAPS inline assembly instructions.

[Yuri Gorshenin]

  PTAL


================
Comment at: lib/Target/X86/AsmParser/X86AsmInstrumentation.h:29
@@ +28,3 @@
+  // instruction is sent to |Out|.
+  virtual bool InstrumentInstruction(
+      const MCInst& Inst,
----------------
Rafael Ávila de Espíndola wrote:
> The bool return is never used. Using |foo| to refer to comments is not very common in llvm.
Done.

================
Comment at: lib/Target/X86/AsmParser/X86AsmInstrumentation.h:39
@@ +38,3 @@
+
+class X86AsmInstrumentationFactory {
+ public:
----------------
Rafael Ávila de Espíndola wrote:
> Why have a class with just a static method in it? Just add a regular function in the llvm namespace.
Done.

================
Comment at: lib/Target/X86/AsmParser/X86AsmInstrumentation.cpp:39
@@ +38,3 @@
+
+class X86StubInstrumentation : public X86AsmInstrumentation {
+public:
----------------
Rafael Ávila de Espíndola wrote:
> Is there value in having a stub implementation instead of making the base a do nothing implementation?
Done.


http://llvm-reviews.chandlerc.com/D2881