[PATCH] Add -fsanitize=leak to driver options.
Sergey Matveev
earthdok at google.com
Thu May 23 07:12:20 PDT 2013
================
Comment at: lib/Driver/Tools.cpp:1605
@@ +1604,3 @@
+ D.Diag(diag::err_drv_argument_not_allowed_with)
+ << lastArgumentForKind(D, Args, Leak)
+ << lastArgumentForKind(D, Args, NeedsTsanRt);
----------------
Alexey Samsonov wrote:
> Why not NeedsLeakDetection here?
ok
================
Comment at: lib/Driver/Tools.cpp:1609
@@ +1608,3 @@
+ D.Diag(diag::err_drv_argument_not_allowed_with)
+ << lastArgumentForKind(D, Args, Leak)
+ << lastArgumentForKind(D, Args, NeedsMsanRt);
----------------
Alexey Samsonov wrote:
> and here
ok
================
Comment at: lib/Driver/Tools.cpp:1718
@@ -1708,4 +1717,3 @@
addSanitizerRTLinkFlagsLinux(TC, Args, CmdArgs, "asan", true);
- }
}
}
----------------
Alexey Samsonov wrote:
> So, what about flipping the default leak detection on/off in presence of -fsanitize=address,leaks?
added a fixme
http://llvm-reviews.chandlerc.com/D837
More information about the cfe-commits
mailing list