[PATCH] clang-cl: support -fsanitize=address

Reid Kleckner rnk at google.com
Tue Aug 27 10:17:39 PDT 2013


  LGTM


================
Comment at: lib/Driver/Tools.cpp:6567
@@ +6566,3 @@
+    llvm::sys::path::append(
+        LibSanitizer, "lib", "windows", "clang_rt.asan-i386.lib");
+    CmdArgs.push_back(Args.MakeArgString(LibSanitizer));
----------------
Timur Iskhodzhanov wrote:
> Do you know an easy way we can add the asan_dll_thunk.lib instead when we build a DLL?
Yep, /LD (or /LDd), but we don't support that flag yet I believe.  IMO it can come when we do.


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



More information about the cfe-commits mailing list