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

Chandler Carruth chandlerc at gmail.com
Fri Nov 30 02:02:17 PST 2012


  One tweak needed to the diagnostics... Otherwise LGTM.


================
Comment at: llvm/tools/clang/include/clang/Basic/DiagnosticDriverKinds.td:104-107
@@ -103,2 +103,6 @@
   "AddressSanitizer on Android requires '-pie'">;
+def err_drv_tsan_requires_pie : Error<
+  "ThreadSanitizer requires '-pie'">;
+def err_drv_msan_requires_pie : Error<
+  "MemorySanitizer requires '-pie'">;
 def err_drv_unknown_objc_runtime : Error<
----------------
You don't need two diagnostics:

  def err_drv_sanitizer_requires_pie : Error<
    "%select{thread|memory}0 sanitizer requires the '-pie' option">;


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



More information about the cfe-commits mailing list