This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] -f{no-sanitize,{,no-}sanitize-recover}=all support

From: Jeff Law <law at redhat dot com>
To: Jakub Jelinek <jakub at redhat dot com>, Richard Biener <rguenther at suse dot de>, Dodji Seketeli <dseketel at redhat dot com>
Cc: Alexey Samsonov <samsonov at google dot com>, Yury Gribov <y dot gribov at samsung dot com>, Marek Polacek <polacek at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Dmitry Vyukov <dvyukov at google dot com>, Konstantin Serebryany <konstantin dot s dot serebryany at gmail dot com>
Date: Mon, 05 Jan 2015 13:32:42 -0700
Subject: Re: [PATCH] -f{no-sanitize,{,no-}sanitize-recover}=all support
Authentication-results: sourceware.org; auth=none
References: <543BADAB dot 4090000 at samsung dot com> <20141017161334 dot GF10376 at tucnak dot redhat dot com> <CAGSYnCMEeTWES+9wzOir6hSXzAdvoQZYWOzd8JURbAfURXhXqA at mail dot gmail dot com> <20141118064740 dot GE1745 at tucnak dot redhat dot com> <CAGSYnCNGEXq=D76sF26G1tP6nKiMkKaC1FJm9KL1svojzXG-vA at mail dot gmail dot com> <20141118075317 dot GF1745 at tucnak dot redhat dot com> <CAGSYnCPW9igmQL_GLoAVc0Jq0vrytHjLvDY8X98kYkZ4=UNEMg at mail dot gmail dot com> <CAGSYnCPYyyWseiJh00f3whDok1gEF39fvxKDUvAB3q-s=8kWpg at mail dot gmail dot com> <20141219080718 dot GS1667 at tucnak dot redhat dot com> <CAGSYnCNLoU0p3FGDwb6mMAAOWFz2Te1m7wmWD94PhcADsQs9rQ at mail dot gmail dot com> <20150105165853 dot GT1667 at tucnak dot redhat dot com>

On 01/05/15 09:58, Jakub Jelinek wrote:

On Tue, Dec 23, 2014 at 04:47:06PM -0800, Alexey Samsonov wrote:

Right, -fsanitize=all produces an error, while -fsanitize-recover=all
enables recovery for all
sanitizers, -fno-sanitize-recover=all disables recovery for everything.


Ok, here is a GCC patch for that.
-fno-sanitize=all is supported as a way to turn off all the sanitizers,
-f{,no-}sanitize-recover=all is supported too, and one can mix all
with other sanitizer names.

Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

2015-01-05  Jakub Jelinek  <jakub@redhat.com>

	* opts.c (common_handle_option): Add support for
	-fno-sanitize=all and -f{,no-}sanitize-recover=all.

	* c-c++-common/asan/sanitize-all-1.c: New test.
	* c-c++-common/ubsan/sanitize-all-1.c: New test.
	* c-c++-common/ubsan/sanitize-all-2.c: New test.
	* c-c++-common/ubsan/sanitize-all-3.c: New test.
	* c-c++-common/ubsan/sanitize-all-4.c: New test.

Are there any doc updates that need to happen as a result of this patch?Patch itself is fine for the trunk, just want to make sure the docside is good too.


jeff

Follow-Ups:
- Re: [PATCH] -f{no-sanitize,{,no-}sanitize-recover}=all support
  - From: Jakub Jelinek

References:
- [PATCH] -f{no-sanitize,{,no-}sanitize-recover}=all support
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]