diff mbox

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

Message ID CAAgBjMkQC4S8UTH686T6AMpZKdjMQ6cv1SQfC-FJ6R_Sg_ydqQ@mail.gmail.com
State New
Headers show

Commit Message

Prathamesh Kulkarni Sept. 2, 2016, 5:06 p.m. UTC 
On 2 September 2016 at 15:49, Richard Biener <rguenther@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.kulkarni@linaro.org>

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

Patch

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;