[PATCH] Asm instrumentation is disabled on non-Linux OSes.

Yuri Gorshenin ygorshenin at chromium.org
Wed Apr 30 06:02:15 PDT 2014


PTAL

================
Comment at: lib/Target/X86/AsmParser/X86AsmInstrumentation.cpp:214
@@ +213,3 @@
+  // FIXME: remove OS check as soon as asm instrumentation functions from
+  // compiler-rt are fixed.
+  Triple T(STI.getTargetTriple());
----------------
Evgeniy Stepanov wrote:
> I don't think they will ever be fixed for all known platforms. Do something like
> bool hasCompilerRTSupport=T.isOSLinux(); and remove the fixme.
> 
Done.

http://reviews.llvm.org/D3564






More information about the llvm-commits mailing list