[cfe-commits] [PATCH] Clang support for MemorySanitizer

Alexey Samsonov samsonov at google.com
Thu Nov 29 09:31:01 PST 2012


  Can you please do svn diff --diff-cmd=diff -x -U999999 for patches you submit to phabricator? (from http://llvm.org/docs/Phabricator.html).
  In this way one can optionally see the full context.


================
Comment at: llvm/tools/clang/lib/Driver/Tools.cpp:4096
@@ -4073,1 +4095,3 @@
+      Sanitize.needsMsanRt() ||
+      Sanitize.needsUbsanRt()) {
     if (Args.hasArg(options::OPT_dynamiclib) ||
----------------
This is Darwin-specific code, right?
Please either remove this (as MSan is not supported on Darwin yet),
or fix lib/Driver/ToolChains.cpp as well.


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



More information about the cfe-commits mailing list