This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [google] AddressSanitizer for gcc, first attempt. (issue 5272048)
- From: dnovillo at google dot com
- To: konstantin dot s dot serebryany at gmail dot com, kcc at google dot com, davidxl at google dot com
- Cc: gcc-patches at gcc dot gnu dot org, reply at codereview dot appspotmail dot com
- Date: Wed, 02 Nov 2011 18:52:53 +0000
- Subject: Re: [google] AddressSanitizer for gcc, first attempt. (issue 5272048)
- Reply-to: konstantin dot s dot serebryany at gmail dot com, kcc at google dot com, davidxl at google dot com, dnovillo at google dot com, gcc-patches at gcc dot gnu dot org, reply at codereview dot appspotmail dot com
OK for google/main with the nits below.
http://codereview.appspot.com/5272048/diff/42003/ChangeLog.google-main
File ChangeLog.google-main (right):
http://codereview.appspot.com/5272048/diff/42003/ChangeLog.google-main#newcode1
ChangeLog.google-main:1: 2011-11-02 Kostya Serebryany
<kcc@google.com>
1 2011-11-02 Kostya Serebryany <kcc@google.com>
Need blank line below the datestamp.
http://codereview.appspot.com/5272048/diff/42003/ChangeLog.google-main#newcode3
ChangeLog.google-main:3: AddressSanitizer, a fast memory error detector:
2 Introduce a new option -fasan which enables
3 AddressSanitizer, a fast memory error detector:
Not strictly accepted, but it is common to add some verbiage of this
nature when adding major features. OK.
http://codereview.appspot.com/5272048/