Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

On 2021/12/29 20:55, Jan Hubicka wrote: >>> >>> From: Xiong Hu Luo >>> >>> gcc/ChangeLog: >>> >>> * loop-invariant.c (find_invariants_bb): Check profile count >>> before motion. >>> (find_invariants_body): Add argument. >>> >>> gcc/testsuite/ChangeLog: >>> >>> * gcc.dg/loop-inva

Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

> > > > From: Xiong Hu Luo > > > > gcc/ChangeLog: > > > > * loop-invariant.c (find_invariants_bb): Check profile count > > before motion. > > (find_invariants_body): Add argument. > > > > gcc/testsuite/ChangeLog: > > > > * gcc.dg/loop-invariant-2.c: New. OK, thanks! Honza

Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

On 2021/12/17 09:30, Xionghu Luo via Gcc-patches wrote: > > > On 2021/12/16 19:20, Jan Hubicka wrote: >>> >>> OK. Comments like? >>> >>> /* Don't move insn of cold BB out of loop to preheader to reduce >>> calculations >>>and register live range in hot loop with cold BB. */ >> >> Looks

Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

On 2021/12/16 19:20, Jan Hubicka wrote: >> >> OK. Comments like? >> >> /* Don't move insn of cold BB out of loop to preheader to reduce calculations >>and register live range in hot loop with cold BB. */ > > Looks good. >> >> >> And maybe some dump log will help tracking in xxx.c.271r.lo

Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

> > OK. Comments like? > > /* Don't move insn of cold BB out of loop to preheader to reduce calculations >and register live range in hot loop with cold BB. */ Looks good. > > > And maybe some dump log will help tracking in xxx.c.271r.loop2_invariant. > > --- a/gcc/loop-invariant.c > +++

Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

On 2021/12/13 18:24, Jan Hubicka wrote: >>> gcc/ChangeLog: >>> >>> * loop-invariant.c (find_invariants_bb): Check profile count >>> before motion. >>> (find_invariants_body): Add argument. >>> --- >>> gcc/loop-invariant.c | 10 +++--- >>> 1 file changed, 7 insertions(+), 3 delet

Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

> > gcc/ChangeLog: > > > > * loop-invariant.c (find_invariants_bb): Check profile count > > before motion. > > (find_invariants_body): Add argument. > > --- > > gcc/loop-invariant.c | 10 +++--- > > 1 file changed, 7 insertions(+), 3 deletions(-) > > > > diff --git a/gcc/loop-inv

Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

> gcc/ChangeLog: > > * loop-invariant.c (find_invariants_bb): Check profile count > before motion. > (find_invariants_body): Add argument. > --- > gcc/loop-invariant.c | 10 +++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/gcc/loop-invariant.c b/gcc

Re: [PATCH 1/3] loop-invariant: Don't move cold bb instructions to preheader in RTL

On 12/7/2021 10:54 PM, Xionghu Luo via Gcc-patches wrote: gcc/ChangeLog: * loop-invariant.c (find_invariants_bb): Check profile count before motion. (find_invariants_body): Add argument. OK jeff