> 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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to