Re: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 denominator

2022-05-23 Thread Richard Biener via Gcc-patches
ner > Sent: Thursday, May 12, 2022 12:34 AM > To: Eugene Rozenfeld > Cc: Jan Hubicka ; gcc-patches@gcc.gnu.org > Subject: Re: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 > denominator > > On Thu, May 12, 2022 at 3:37 AM Eugene Rozenfeld > wrote: > >

RE: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 denominator

2022-05-20 Thread Eugene Rozenfeld via Gcc-patches
t: Re: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 denominator On Thu, May 12, 2022 at 3:37 AM Eugene Rozenfeld wrote: > > In my case this is not exactly what the FIXME in the comment above > says. The count is 0 even before the initial scaling happens. I hit this cas

Re: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 denominator

2022-05-12 Thread Richard Biener via Gcc-patches
_vector) is not zero instead of checking that bbs[i].count is not zero? Richard. > -Original Message- > From: Richard Biener > Sent: Monday, May 09, 2022 12:42 AM > To: Eugene Rozenfeld ; Jan Hubicka > > Cc: gcc-patches@gcc.gnu.org > Subject: [EXTERNAL] Re:

RE: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 denominator

2022-05-11 Thread Eugene Rozenfeld via Gcc-patches
es@gcc.gnu.org Subject: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 denominator On Fri, May 6, 2022 at 10:32 PM Eugene Rozenfeld via Gcc-patches wrote: > > Calling count.apply_scale with a 0 denominator causes an assert. > This change guards against that. > > Tested on x

Re: [PATCH] Guard against applying scale with 0 denominator

2022-05-09 Thread Richard Biener via Gcc-patches
On Fri, May 6, 2022 at 10:32 PM Eugene Rozenfeld via Gcc-patches wrote: > > Calling count.apply_scale with a 0 denominator causes an assert. > This change guards against that. > > Tested on x86_64-pc-linux-gnu. > > gcc/ChangeLog: > * tree-loop-vect-manip.cc (vect_do_peeling): Guard against

[PATCH] Guard against applying scale with 0 denominator

2022-05-06 Thread Eugene Rozenfeld via Gcc-patches
Calling count.apply_scale with a 0 denominator causes an assert. This change guards against that. Tested on x86_64-pc-linux-gnu. gcc/ChangeLog: * tree-loop-vect-manip.cc (vect_do_peeling): Guard against applying scale with 0 denominator. --- gcc/tree-vect-loop-manip.cc | 9 + 1