Patchwork [00/13] Request to merge Address Sanitizer in

login
register
mail settings
Submitter Tobias Burnus
Date Nov. 12, 2012, 4:45 p.m.
Message ID <50A12797.6030100@net-b.de>
Download mbox | patch
Permalink /patch/198418/
State New
Headers show

Comments

Tobias Burnus - Nov. 12, 2012, 4:45 p.m.
Jakub Jelinek:
> On Mon, Nov 12, 2012 at 05:07:42PM +0100, Dodji Seketeli wrote:
>> Following a request from Jakub, and given the fact that the patch set
>> have been reviewed by Diego, I have committed the last set of patches I
>> have posted to trunk.
>
> Thanks, I've committed as obvious the following formatting cleanup.

Also thanks from my side!

I have also committed a small patch, which restores the single "-f…" 
(instead of "--f…" vs. "f…" and which moved -faddress-sanitizer from the 
optimization to the debugging option section.

Tobias
Konstantin Serebryany - Nov. 12, 2012, 4:50 p.m.
Folks, please remember that the Clang flag has recently changed to
-fsanitize=address (-fsanitize=thread).
This is hopefully the last syntax change there.

--kcc

On Mon, Nov 12, 2012 at 8:45 AM, Tobias Burnus <burnus@net-b.de> wrote:
> Jakub Jelinek:
>
>> On Mon, Nov 12, 2012 at 05:07:42PM +0100, Dodji Seketeli wrote:
>>>
>>> Following a request from Jakub, and given the fact that the patch set
>>> have been reviewed by Diego, I have committed the last set of patches I
>>> have posted to trunk.
>>
>>
>> Thanks, I've committed as obvious the following formatting cleanup.
>
>
> Also thanks from my side!
>
> I have also committed a small patch, which restores the single "-f…"
> (instead of "--f…" vs. "f…" and which moved -faddress-sanitizer from the
> optimization to the debugging option section.
>
> Tobias

Patch

Index: gcc/ChangeLog
===================================================================
--- gcc/ChangeLog	(Revision 193442)
+++ gcc/ChangeLog	(Arbeitskopie)
@@ -1,3 +1,8 @@ 
+2012-11-12  Tobias Burnus  <burnus@net-b.de>
+
+	* doc/invoke.texi: Move -faddress-sanitizer from Optimization
+	Options to Debugging Options.
+
 2012-11-12  Jakub Jelinek  <jakub@redhat.com>
 
 	* asan.c: Formatting cleanups.
@@ -164,7 +169,7 @@ 
 
 	* Makefile.in: Add asan.c and its dependencies.
 	* common.opt: Add -faddress-sanitizer option.
-	* invoke.texi: Document the new flag.
+	* doc/invoke.texi: Document the new flag.
 	* passes.c: Add the asan pass.
 	* toplev.c (compile_file): Call asan_finish_file.
 	* asan.c: New file.
Index: gcc/doc/invoke.texi
===================================================================
--- gcc/doc/invoke.texi	(Revision 193442)
+++ gcc/doc/invoke.texi	(Arbeitskopie)
@@ -289,7 +289,7 @@  Objective-C and Objective-C++ Dialects}.
 @item Debugging Options
 @xref{Debugging Options,,Options for Debugging Your Program or GCC}.
 @gccoptlist{-d@var{letters}  -dumpspecs  -dumpmachine  -dumpversion @gol
--fdbg-cnt-list -fdbg-cnt=@var{counter-value-list} @gol
+-faddress-sanitizer -fdbg-cnt-list -fdbg-cnt=@var{counter-value-list} @gol
 -fdisable-ipa-@var{pass_name} @gol
 -fdisable-rtl-@var{pass_name} @gol
 -fdisable-rtl-@var{pass-name}=@var{range-list} @gol
@@ -354,10 +354,10 @@  Objective-C and Objective-C++ Dialects}.
 @item Optimization Options
 @xref{Optimize Options,,Options that Control Optimization}.
 @gccoptlist{-falign-functions[=@var{n}] -falign-jumps[=@var{n}] @gol
--falign-labels[=@var{n}] -falign-loops[=@var{n}] -faddress-sanitizer @gol
---fassociative-math fauto-inc-dec -fbranch-probabilities @gol
---fbranch-target-load-optimize fbranch-target-load-optimize2 @gol
---fbtr-bb-exclusive -fcaller-saves @gol
+-falign-labels[=@var{n}] -falign-loops[=@var{n}] @gol
+-fassociative-math -fauto-inc-dec -fbranch-probabilities @gol
+-fbranch-target-load-optimize -fbranch-target-load-optimize2 @gol
+-fbtr-bb-exclusive -fcaller-saves @gol
 -fcheck-data-deps -fcombine-stack-adjustments -fconserve-stack @gol
 -fcompare-elim -fcprop-registers -fcrossjumping @gol
 -fcse-follow-jumps -fcse-skip-blocks -fcx-fortran-rules @gol