Re: [PATCH] rs6000: Adjust loop_unroll_adjust to match middle-end change [PR 107692]

2022-11-23 Thread Richard Biener via Gcc-patches
On Wed, Nov 23, 2022 at 2:53 AM Hongyu Wang wrote: > > Hi, Segher and Richard > > > > Something in your patch was wrong, please fix that (or revert the > > > patch). You should not have to touch config/rs6000/ at all. > > > > Sure something is wrong, but I think there's the opportunity to > > sim

Re: [PATCH] rs6000: Adjust loop_unroll_adjust to match middle-end change [PR 107692]

2022-11-22 Thread Hongyu Wang via Gcc-patches
Hi, Segher and Richard > > Something in your patch was wrong, please fix that (or revert the > > patch). You should not have to touch config/rs6000/ at all. > > Sure something is wrong, but I think there's the opportunity to > simplify rs6000/ and s390x/, the only other two implementors of > the

Re: [PATCH] rs6000: Adjust loop_unroll_adjust to match middle-end change [PR 107692]

2022-11-22 Thread Richard Biener via Gcc-patches
On Fri, Nov 18, 2022 at 3:47 PM Segher Boessenkool wrote: > > [ Please cc: me and Ke Wen on rs6000 patches ] > > On Thu, Nov 17, 2022 at 07:54:29AM +0800, Hongyu Wang wrote: > > r13-3950-g071e428c24ee8c enables O2 small loop unrolling, but it breaks > > -fno-unroll-loops for rs6000 with loop_unrol

Re: [PATCH] rs6000: Adjust loop_unroll_adjust to match middle-end change [PR 107692]

2022-11-18 Thread Segher Boessenkool
[ Please cc: me and Ke Wen on rs6000 patches ] On Thu, Nov 17, 2022 at 07:54:29AM +0800, Hongyu Wang wrote: > r13-3950-g071e428c24ee8c enables O2 small loop unrolling, but it breaks > -fno-unroll-loops for rs6000 with loop_unroll_adjust hook. Adjust the > option handling and target hook accordingl

[PATCH] rs6000: Adjust loop_unroll_adjust to match middle-end change [PR 107692]

2022-11-16 Thread Hongyu Wang via Gcc-patches
Hi, r13-3950-g071e428c24ee8c enables O2 small loop unrolling, but it breaks -fno-unroll-loops for rs6000 with loop_unroll_adjust hook. Adjust the option handling and target hook accordingly. Bootstrapped & regtested on powerpc64le-linux-gnu, OK for trunk? gcc/ChangeLog: PR target/107692