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

Evgeniy Stepanov eugenis at google.com
Wed Apr 30 05:38:01 PDT 2014


================
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());
----------------
I don't think they will ever be fixed for all known platforms. Do something like
bool hasCompilerRTSupport=T.isOSLinux(); and remove the fixme.

http://reviews.llvm.org/D3564






More information about the llvm-commits mailing list