Segher Boessenkool writes:
> Hi!
>
> On Thu, May 28, 2020 at 04:22:16PM +0200, Richard Biener wrote:
>> For GIMPLE level transforms I don't think targets have more knowledge
>> than the middle-end.
>
> Yes, certainly.
>
>> In fact GIMPLE complete unrolling is about
>> secondary effects, removing
Hi!
On Thu, May 28, 2020 at 04:22:16PM +0200, Richard Biener wrote:
> For GIMPLE level transforms I don't think targets have more knowledge
> than the middle-end.
Yes, certainly.
> In fact GIMPLE complete unrolling is about
> secondary effects, removing redundancies and abstraction. So IMHO
> t
On Tue, May 26, 2020 at 6:58 AM Jiufu Guo wrote:
>
> David Edelsohn writes:
>
> > On Mon, May 25, 2020 at 1:58 PM Richard Biener
> > wrote:
> >>
> >> On May 25, 2020 7:40:00 PM GMT+02:00, Segher Boessenkool
> >> wrote:
> >> >On Mon, May 25, 2020 at 02:14:02PM +0200, Richard Biener wrote:
> >>
David Edelsohn writes:
> On Mon, May 25, 2020 at 1:58 PM Richard Biener
> wrote:
>>
>> On May 25, 2020 7:40:00 PM GMT+02:00, Segher Boessenkool
>> wrote:
>> >On Mon, May 25, 2020 at 02:14:02PM +0200, Richard Biener wrote:
>> >> On Mon, May 25, 2020 at 1:10 PM guojiufu
>> >wrote:
>> >> Since a
On Mon, May 25, 2020 at 07:58:09PM +0200, Richard Biener wrote:
> You folks made ppc specific hacks instead of a better design. Those now stand
> in the way as well. But sure, simply do not expose the flag to the users, use
> Var(flag_rtl_unroll_loops). My other points still stand.
>
> Feel free
On Mon, May 25, 2020 at 1:58 PM Richard Biener
wrote:
>
> On May 25, 2020 7:40:00 PM GMT+02:00, Segher Boessenkool
> wrote:
> >On Mon, May 25, 2020 at 02:14:02PM +0200, Richard Biener wrote:
> >> On Mon, May 25, 2020 at 1:10 PM guojiufu
> >wrote:
> >> Since a new flag is not needed to fix the r
On May 25, 2020 7:40:00 PM GMT+02:00, Segher Boessenkool
wrote:
>On Mon, May 25, 2020 at 02:14:02PM +0200, Richard Biener wrote:
>> On Mon, May 25, 2020 at 1:10 PM guojiufu
>wrote:
>> Since a new flag is not needed to fix the regression please avoid
>> adding -fcomplete-unroll-loops.
>>
>> For
On Mon, May 25, 2020 at 02:14:02PM +0200, Richard Biener wrote:
> On Mon, May 25, 2020 at 1:10 PM guojiufu wrote:
> Since a new flag is not needed to fix the regression please avoid
> adding -fcomplete-unroll-loops.
>
> For -frtl-unroll-loops you should be able to use
Erm. That *is* a new comma
On Mon, May 25, 2020 at 1:10 PM guojiufu wrote:
>
> Currently option -funroll-loops controls both GIMPLE unroler and
> RTL unroller. It is not able to control GIMPLE cunroller and
> RTL unroller independently. This patch introducing different flags
> to control them seperately, and this also prov
