Re: [PATCH 10/10] vect: Reuse reduction accumulators between loops

2021-07-12 Thread Richard Biener via Gcc-patches
On Mon, Jul 12, 2021 at 7:55 PM Richard Sandiford wrote: > > Richard Biener via Gcc-patches writes: > > On Fri, Jul 9, 2021 at 3:12 PM Richard Sandiford > > wrote: > >> > >> Thanks for the review. > >> > >> Richard Biener writes: > >> >> @@ -588,6 +600,23 @@ public: > >> >>/* Unrolling fact

Re: [PATCH 10/10] vect: Reuse reduction accumulators between loops

2021-07-12 Thread Richard Sandiford via Gcc-patches
Richard Biener via Gcc-patches writes: > On Fri, Jul 9, 2021 at 3:12 PM Richard Sandiford > wrote: >> >> Thanks for the review. >> >> Richard Biener writes: >> >> @@ -588,6 +600,23 @@ public: >> >>/* Unrolling factor */ >> >>poly_uint64 vectorization_factor; >> >> >> >> + /* If this lo

Re: [PATCH 10/10] vect: Reuse reduction accumulators between loops

2021-07-11 Thread Richard Biener via Gcc-patches
On Fri, Jul 9, 2021 at 3:12 PM Richard Sandiford wrote: > > Thanks for the review. > > Richard Biener writes: > >> @@ -588,6 +600,23 @@ public: > >>/* Unrolling factor */ > >>poly_uint64 vectorization_factor; > >> > >> + /* If this loop is an epilogue loop whose main loop can be skipped

Re: [PATCH 10/10] vect: Reuse reduction accumulators between loops

2021-07-09 Thread Richard Sandiford via Gcc-patches
Thanks for the review. Richard Biener writes: >> @@ -588,6 +600,23 @@ public: >>/* Unrolling factor */ >>poly_uint64 vectorization_factor; >> >> + /* If this loop is an epilogue loop whose main loop can be skipped, >> + MAIN_LOOP_EDGE is the edge from the main loop to this loop's >>

Re: [PATCH 10/10] vect: Reuse reduction accumulators between loops

2021-07-09 Thread Richard Biener via Gcc-patches
On Thu, Jul 8, 2021 at 2:50 PM Richard Sandiford via Gcc-patches wrote: > > This patch adds support for reusing a main loop's reduction accumulator > in an epilogue loop. This in turn lets the loops share a single piece > of vector->scalar reduction code. > > The patch has the following restricti