[PATCH] clang-cl: Diagnose the usage of ASAN with a debug runtime library

Ehsan Akhgari ehsan.akhgari at gmail.com
Mon Oct 13 17:15:58 PDT 2014


================
Comment at: lib/Driver/Tools.cpp:3590
@@ +3589,3 @@
+                                 Args.hasArg(options::OPT__SLASH_LDd))) {
+    D.Diag(diag::err_drv_address_sanitizer_debug_runtime);
+  }
----------------
samsonov wrote:
> Why not use err_drv_argument_not_allowed_with here and show that "-fsanitize=address not allowed with /MTd" or smth. like that?
Well, technically there is nothing wrong with using these two arguments together.  I tried to make it clear in the diagnostic that this is something that is just not supported yet.  Do you think that's reasonable?

http://reviews.llvm.org/D5764






More information about the cfe-commits mailing list