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

2016-09-02 Thread Prathamesh Kulkarni
On 2 September 2016 at 15:49, Richard Biener 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

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

2016-09-02 Thread Richard Biener
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://

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

2016-09-02 Thread Prathamesh Kulkarni
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 Thanks, Prathamesh diff