[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