Re: cfg.c: redundant second assignment of bb_copy = NULL in free_original_copy_tables()

[Prathamesh Kulkarni]

On 2 September 2016 at 15:49, Richard Biener <rguent...@suse.de> wrote:
> On Fri, 2 Sep 2016, Prathamesh Kulkarni wrote:
>
>> Hi,
>> There appears to be a redundant second assignmeent bb_copy = NULL in
>> free_copy_original_tables(). I suppose it should be
>> bb_original = NULL instead ?
>> I found this mentioned on a blog "Bugs found in gcc with help of PVS studio":
>> http://www.viva64.com/en/b/0425/#ID0EHCCK
>
> Ok.
Thanks, committed as r239960 after bootstrap+test on x86_64-unknown-linux-gnu.

Regards,
Prathamesh
>
> Thanks,
> Richard.

2016-09-02  Prathamesh Kulkarni  <prathamesh.kulka...@linaro.org>

        * cfg.c (free_original_copy_tables): Replace second assignment of
        bb_copy = NULL by bb_original = NULL. 

diff --git a/gcc/cfg.c b/gcc/cfg.c
index 0e31780..cab66c6 100644
--- a/gcc/cfg.c
+++ b/gcc/cfg.c
@@ -1075,7 +1075,7 @@ free_original_copy_tables (void)
   delete bb_copy;
   bb_copy = NULL;
   delete bb_original;
-  bb_copy = NULL;
+  bb_original = NULL;
   delete loop_copy;
   loop_copy = NULL;
   delete original_copy_bb_pool;