This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] -f{no-sanitize,{,no-}sanitize-recover}=all support
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: Richard Biener <rguenther at suse dot de>, Dodji Seketeli <dseketel at redhat dot com>, 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, 5 Jan 2015 22:40:37 +0100
- Subject: Re: [PATCH] -f{no-sanitize,{,no-}sanitize-recover}=all support
- Authentication-results: sourceware.org; auth=none
- References: <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> <54AAF4EA dot 2060208 at redhat dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Jan 05, 2015 at 01:32:42PM -0700, Jeff Law wrote:
> 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 doc side is
> good too.
You're right, I'll add documentation tomorrow and repost the patch.
Jakub