[PATCH] [asan] Update the llvm webpage describing ASan

Anna Zaks zaks.anna at gmail.com
Fri Jun 19 16:51:37 PDT 2015


================
Comment at: docs/AddressSanitizer.rst:66
@@ +65,3 @@
+faster (both by ~5%). Another reason: this makes fixing bugs unavoidable. With
+Valgrind, it is often the case that users treat Valgrind warnings as false
+positives (which they are not) and don't fix them.
----------------
samsonov wrote:
> I think we don't have to mention Valgrind, and just claim that all ASan errors are true positives, and
> the program can be in inconsistent state, causing confusing results, and (potentially) misleading
> subsequent reports.
Will do. This was copy and pasted from the original.

================
Comment at: docs/AddressSanitizer.rst:228
@@ -186,3 +227,3 @@
 * Linux i386/x86\_64 (tested on Ubuntu 12.04);
-* MacOS 10.6 - 10.9 (i386/x86\_64).
+* OS X 10.6 - 10.11 (i386/x86\_64).
 * Android ARM
----------------
samsonov wrote:
> I wonder if we can bump it to 10.7...
I'd definitely be ok with this!

================
Comment at: docs/AddressSanitizer.rst:256
@@ +255,3 @@
+
+For the experimental memory leak detector in AddressSanitizer, see
+:doc:`LeakSanitizer`.
----------------
samsonov wrote:
> While you're here.... it's not experimental anymore - it's on by default for Linux. (not implemented on Mac)
What about Android and FreeBSD? Is it only available on Linux?

http://reviews.llvm.org/D10559

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list