cfg.c: redundant second assignment of bb_copy = NULL in free_original_copy_tables()
Prathamesh Kulkarni
prathamesh.kulkarni@linaro.org
Fri Sep 2 17:07:00 GMT 2016
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.
-------------- next part --------------
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 --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;
More information about the Gcc-patches
mailing list