Re: PR80806

2017-06-30 Thread Jakub Jelinek
On Fri, Jun 30, 2017 at 10:25:51AM +0200, Richard Biener wrote: > Do not go down the route of -Wunreachable-code again please. Yeah, I don't think we want -Wunused-but-set* as a late warning, it is intentionally in the FE where is the only place where the false positive ratio of the warning can st

Re: PR80806

2017-06-30 Thread Richard Biener
On Thu, 29 Jun 2017, Jeff Law wrote: > On 05/18/2017 12:55 PM, Prathamesh Kulkarni wrote: > > Hi, > > The attached patch tries to fix PR80806 by warning when a variable is > > set using memset (and friends) but not used. I chose to warn in dse > > pass since dse would detect if the variable passed

Re: PR80806

2017-06-29 Thread Martin Sebor
On 06/29/2017 12:05 PM, Jeff Law wrote: On 06/29/2017 11:57 AM, Jeff Law wrote: On 05/23/2017 09:58 AM, Martin Sebor wrote: On 05/18/2017 12:55 PM, Prathamesh Kulkarni wrote: Hi, The attached patch tries to fix PR80806 by warning when a variable is set using memset (and friends) but not used.

Re: PR80806

2017-06-29 Thread Jeff Law
On 05/18/2017 12:55 PM, Prathamesh Kulkarni wrote: > Hi, > The attached patch tries to fix PR80806 by warning when a variable is > set using memset (and friends) but not used. I chose to warn in dse > pass since dse would detect if the variable passed as 1st argument is > a dead store. Does this ap

Re: PR80806

2017-06-29 Thread Jeff Law
On 06/29/2017 11:57 AM, Jeff Law wrote: > On 05/23/2017 09:58 AM, Martin Sebor wrote: >> On 05/18/2017 12:55 PM, Prathamesh Kulkarni wrote: >>> Hi, >>> The attached patch tries to fix PR80806 by warning when a variable is >>> set using memset (and friends) but not used. I chose to warn in dse >>> p

Re: PR80806

2017-06-29 Thread Jeff Law
On 05/23/2017 09:58 AM, Martin Sebor wrote: > On 05/18/2017 12:55 PM, Prathamesh Kulkarni wrote: >> Hi, >> The attached patch tries to fix PR80806 by warning when a variable is >> set using memset (and friends) but not used. I chose to warn in dse >> pass since dse would detect if the variable pass

Re: PR80806

2017-05-23 Thread Martin Sebor
I attach an updated patch that actually bootstraps and (with one minor exception) passes regression tests. It's a C front end-only proof of concept for now. The complete patch will include attributes read-only and read-write and support C++ of course. I think the optimization bits can be done i

Re: PR80806

2017-05-23 Thread Martin Sebor
On 05/18/2017 12:55 PM, Prathamesh Kulkarni wrote: Hi, The attached patch tries to fix PR80806 by warning when a variable is set using memset (and friends) but not used. I chose to warn in dse pass since dse would detect if the variable passed as 1st argument is a dead store. Does this approach l

Re: PR80806

2017-05-23 Thread Prathamesh Kulkarni
On 22 May 2017 at 10:03, Jeff Law wrote: > On 05/18/2017 12:55 PM, Prathamesh Kulkarni wrote: >> Hi, >> The attached patch tries to fix PR80806 by warning when a variable is >> set using memset (and friends) but not used. I chose to warn in dse >> pass since dse would detect if the variable passed

Re: PR80806

2017-05-21 Thread Jeff Law
On 05/18/2017 12:55 PM, Prathamesh Kulkarni wrote: > Hi, > The attached patch tries to fix PR80806 by warning when a variable is > set using memset (and friends) but not used. I chose to warn in dse > pass since dse would detect if the variable passed as 1st argument is > a dead store. Does this ap