Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-18 Thread Richard Biener
On Tue, Jan 17, 2017 at 3:42 PM, Jeff Law wrote: > On 01/17/2017 02:15 AM, Richard Biener wrote: >> >> On Mon, Jan 16, 2017 at 11:36 PM, Richard Biener >> wrote: >>> >>> On January 16, 2017 7:27:53 PM GMT+01:00, Jeff Law >>> wrote: On 01/16/2017 01:51 AM, Richard Biener wrote: > >>

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-17 Thread Jeff Law
On 01/17/2017 02:15 AM, Richard Biener wrote: On Mon, Jan 16, 2017 at 11:36 PM, Richard Biener wrote: On January 16, 2017 7:27:53 PM GMT+01:00, Jeff Law wrote: On 01/16/2017 01:51 AM, Richard Biener wrote: On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: At one time I know I had the max_

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-17 Thread Richard Biener
On Mon, Jan 16, 2017 at 11:36 PM, Richard Biener wrote: > On January 16, 2017 7:27:53 PM GMT+01:00, Jeff Law wrote: >>On 01/16/2017 01:51 AM, Richard Biener wrote: >>> On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: At one time I know I had the max_size == size test in >>valid_ao_ref

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-16 Thread Richard Biener
On January 16, 2017 7:27:53 PM GMT+01:00, Jeff Law wrote: >On 01/16/2017 01:51 AM, Richard Biener wrote: >> On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: >>> >>> At one time I know I had the max_size == size test in >valid_ao_ref_for_dse. >>> But it got lost at some point. This is what cause

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-16 Thread Jeff Law
On 01/16/2017 01:51 AM, Richard Biener wrote: On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: At one time I know I had the max_size == size test in valid_ao_ref_for_dse. But it got lost at some point. This is what caused the Ada failure. Technically it'd be OK for the potentially dead stor

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-16 Thread Richard Biener
On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: > > At one time I know I had the max_size == size test in valid_ao_ref_for_dse. > But it got lost at some point. This is what caused the Ada failure. > > Technically it'd be OK for the potentially dead store to have a variable > size as long as th

[PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-15 Thread Jeff Law
At one time I know I had the max_size == size test in valid_ao_ref_for_dse. But it got lost at some point. This is what caused the Ada failure. Technically it'd be OK for the potentially dead store to have a variable size as long as the later stores covered the entire range of the potenti