[PATCH] DataFlowSanitizer; Clang changes.
Kostya Serebryany
kcc at google.com
Mon Jun 17 06:32:19 PDT 2013
LGTM with a small nit.
I am not sure if I have the authority to approve it though.
================
Comment at: lib/Driver/Tools.cpp:1773
@@ +1772,3 @@
+ ArgStringList &CmdArgs) {
+ if (!Args.hasArg(options::OPT_shared)) {
+ addSanitizerRTLinkFlagsLinux(TC, Args, CmdArgs, "dfsan", true);
----------------
remove { }
================
Comment at: docs/DataFlowSanitizerDesign.rst:1
@@ +1,2 @@
+DataFlowSanitizer Design Document
+=================================
----------------
It might be good to give an example use case, the one you replied in the mail discussion:
"take a large application and use this instrumentation as a tool to help monitor how data flows from its inputs (sources) to its outputs (sinks)".
You could add this later.
http://llvm-reviews.chandlerc.com/D966
More information about the cfe-commits
mailing list