On Fri, Nov 11, 2016 at 02:44:40PM +0100, Martin Liška wrote:
> Hello.
>
> Due to a stupid mistake I did, following patch is needed for the test-case
> to properly save previous gimplify_ctxp->live_switch_vars.
>
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
> I was able to run asan bootstrap on x86_64-linux-gnu and kernel build with
> allyesconfig works fine.
>
> Ready to be installed?
> Martin
> >From 53dd3c035283863a25a24feb90bf359295999bca Mon Sep 17 00:00:00 2001
> From: marxin <mli...@suse.cz>
> Date: Fri, 11 Nov 2016 11:21:29 +0100
> Subject: [PATCH] Fix PR sanitizer/78270 (part 2)
>
> gcc/ChangeLog:
>
> 2016-11-11 Martin Liska <mli...@suse.cz>
>
> PR sanitizer/78270
> * gimplify.c (gimplify_switch_expr): Always save previous
> gimplify_ctxp->live_switch_vars.
>
> gcc/testsuite/ChangeLog:
>
> 2016-11-11 Martin Liska <mli...@suse.cz>
>
> PR sanitizer/78270
> * gcc.dg/asan/pr78270-2.c: New test.
Ok, thanks.
> --- /dev/null
> +++ b/gcc/testsuite/gcc.dg/asan/pr78270-2.c
> @@ -0,0 +1,17 @@
> +// { dg-do compile }
> +// { dg-additional-options "-Wno-switch-unreachable" }
Please use /* */ comments instead of //, that is common
in the gcc.dg/ tests, and start with
/* PR sanitizer/78270 */
line.
Thanks.
Jakub
