[PATCH] Add compiler support for dynamic Asan runtime

Alexey Samsonov samsonov at google.com
Wed Mar 26 11:41:06 PDT 2014



================
Comment at: lib/Driver/SanitizerArgs.cpp:174
@@ -172,1 +173,3 @@
+  if (NeedsAsan) {
+    AsanSharedRuntime = Args.hasArg(options::OPT_shared_libasan);
     AsanZeroBaseShadow =
----------------
Why not define AsanSharedRuntime to true for llvm::Triple::Android here?

================
Comment at: lib/Driver/Tools.cpp:1822
@@ +1821,3 @@
+  // or "libclang_rt.<Sanitizer>-dynamic-<ArchName>.so
+  const char *EnvSuffix = 
+    TC.getTriple().getEnvironment() == llvm::Triple::Android ?
----------------
clang-format-diff this diff as well.


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



More information about the llvm-commits mailing list