On 11/16/2016 12:35 PM, Jakub Jelinek wrote:
> 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
> 

Ok, I did the same also for pr78270.c file, patch is installed as r242485.

Thanks,
Martin

Reply via email to