Re: PR85817

2018-05-18 Thread Jeff Law
On 05/18/2018 02:49 AM, Prathamesh Kulkarni wrote: > Hi, > In r260250, the condition > > if (integer_zerop (retval)) > continue; > > was added before checking retval was of pointer type which caused > functions having return type apart from void *, to be marked as > malloc. The attached patch g

Re: PR85817

2018-05-18 Thread Jeff Law
On 05/18/2018 02:49 AM, Prathamesh Kulkarni wrote: > Hi, > In r260250, the condition > > if (integer_zerop (retval)) > continue; > > was added before checking retval was of pointer type which caused > functions having return type apart from void *, to be marked as > malloc. The attached patch g

Re: PR85817

2018-05-18 Thread Richard Biener
On Fri, 18 May 2018, Prathamesh Kulkarni wrote: > Hi, > In r260250, the condition > > if (integer_zerop (retval)) > continue; > > was added before checking retval was of pointer type which caused > functions having return type apart from void *, to be marked as > malloc. The attached patch get

PR85817

2018-05-18 Thread Prathamesh Kulkarni
Hi, In r260250, the condition if (integer_zerop (retval)) continue; was added before checking retval was of pointer type which caused functions having return type apart from void *, to be marked as malloc. The attached patch gets rid of the above check since we do not wish to mark function retu