Re: [RFA][PATCH 3/4] Trim mem* calls in DSE

2017-01-12 Thread Jeff Law
Richi indicated he wants this to be included in gcc-7. So I've updated the mem* trimming patch with his comments. The unnecessary SSA updates are gone. Use wide_int_to_tree rather than building/folding expressions we know will result in constants. Simplify incrementing the start address b

Re: [RFA][PATCH 3/4] Trim mem* calls in DSE

2017-01-12 Thread Jeff Law
On 01/04/2017 07:04 AM, Richard Biener wrote: * tree-ssa-dse.c (need_ssa_update): New file scoped boolean. (decrement_count): New function. (increment_start_addr, trim_memstar_call): Likewise. (trim_partially_dead_store): Call trim_memstar_call. (pass_ds

Re: [RFA][PATCH 3/4] Trim mem* calls in DSE

2017-01-11 Thread Jeff Law
On 01/04/2017 07:04 AM, Richard Biener wrote: Didn't see a re-post of this one so reviewing the old. Didn't figure mem* trimming was suitable for gcc-7 as I couldn't justify it as a bugfix, so I didn't ping it. I don't think it changed materially. All your comments are still applicable to t

Re: [RFA][PATCH 3/4] Trim mem* calls in DSE

2017-01-04 Thread Richard Biener
On Fri, Dec 16, 2016 at 2:54 AM, Jeff Law wrote: > This is the 3rd patch in the kit to improve our DSE implementation. > > This patch supports trimming of the head or tail of a memset, memcpy or > memmove call. It's conceptually similar to trimming CONSTRUCTORS (and was > in fact developed first)

[RFA][PATCH 3/4] Trim mem* calls in DSE

2016-12-15 Thread Jeff Law
This is the 3rd patch in the kit to improve our DSE implementation. This patch supports trimming of the head or tail of a memset, memcpy or memmove call. It's conceptually similar to trimming CONSTRUCTORS (and was in fact developed first). Try as I might, I couldn't find a BZ in our database