Re: [RFA][PATCH 1b/4] [PR tree-optimization/33562] Improve DSE of complex stores

2017-05-05 Thread Jeff Law
On 05/05/2017 08:04 AM, Richard Sandiford wrote: It was primarily to avoid mucking up alignments of the start of the copy or leaving residuals at the end of a copy. It's an idea I saw while scanning the LLVM implementation of DSE. The fact that it avoids mucking things up for tree-ssa-strlen wa

Re: [RFA][PATCH 1b/4] [PR tree-optimization/33562] Improve DSE of complex stores

2017-05-05 Thread Richard Sandiford
Jeff Law writes: > On 05/05/2017 06:13 AM, Richard Sandiford wrote: >> Hi Jeff, >> >> Jeff Law writes: >>> +/* Compute the number of elements that we can trim from the head and >>> + tail of ORIG resulting in a bitmap that is a superset of LIVE. >>> + >>> + Store the number of elements trimm

Re: [RFA][PATCH 1b/4] [PR tree-optimization/33562] Improve DSE of complex stores

2017-05-05 Thread Jeff Law
On 05/05/2017 06:13 AM, Richard Sandiford wrote: Hi Jeff, Jeff Law writes: +/* Compute the number of elements that we can trim from the head and + tail of ORIG resulting in a bitmap that is a superset of LIVE. + + Store the number of elements trimmed from the head and tail in + TRIM_HEAD

Re: [RFA][PATCH 1b/4] [PR tree-optimization/33562] Improve DSE of complex stores

2017-05-05 Thread Richard Sandiford
Hi Jeff, Jeff Law writes: > +/* Compute the number of elements that we can trim from the head and > + tail of ORIG resulting in a bitmap that is a superset of LIVE. > + > + Store the number of elements trimmed from the head and tail in > + TRIM_HEAD and TRIM_TAIL. */ > + > +static void > +

[RFA][PATCH 1b/4] [PR tree-optimization/33562] Improve DSE of complex stores

2017-01-12 Thread Jeff Law
No major changes in this patch, just addressing the small changes requested by Richi. ie, missing docs, dropped some unnecessary prototype movement, improve check in valid_ao_ref_for_dse, minor formatting fixes. I'm going to assume Richi's prior approval still holds as nothing significant