On Thu, Jun 11, 2020 at 9:04 PM Segher Boessenkool
<seg...@kernel.crashing.org> wrote:
>
> Hi!
>
> On Thu, Jun 11, 2020 at 01:49:57PM +0200, Martin Liška wrote:
> > Since stmt_vec_info is not generated for debug info statements, these needs
> > to
> > be skipped in rs6000_density_test.
>
> > PR target/95627
> > * config/rs6000/rs6000.c (rs6000_density_test): Skip debug
> > statements.
>
> > diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
> > index 0bbd06ad1de..00daf979856 100644
> > --- a/gcc/config/rs6000/rs6000.c
> > +++ b/gcc/config/rs6000/rs6000.c
> > @@ -4987,6 +4987,9 @@ rs6000_density_test (rs6000_cost_data *data)
> > for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
> > {
> > gimple *stmt = gsi_stmt (gsi);
> > + if (is_gimple_debug (stmt))
> > + continue;
>
> Yes, this is fine for trunk (with the indent fixed), thanks!
>
> Or you could use gsi_next_nondebug? That is a bit neater. Either way
> is okay.
Also need to adjust gsi_start_bb then, of course.
>
> Thanks!
>
>
> Segher
