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

Hans Wennborg hans at chromium.org
Tue Aug 27 11:12:34 PDT 2013


  > LGTM

  Thanks!


================
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));
----------------
Reid Kleckner wrote:
> 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.
I'll add this to the FIXME.


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



More information about the cfe-commits mailing list