On Fri, 11 Oct 2019, Bernd Edlinger wrote: > Hi Richard, > > I became aware of this while looking at the -Wshadow=compatible-local > warnings. > The function vectorizable_live_operation uses a parameter called "vec_stmt" > that is shadowed by something also called "vec_stmt". But in this case, > the vec_stmt actually only used as a boolean, i.e. pointer is NULL or not. > > This changes the parameter vec_stmt to vec_stmt_p, and propagates that > change to can_vectorize_live_stmts. > > > Bootstrapped and reg-tested on x86_64-pc-linux-gnu. > Is it OK for trunk?
OK. Some more refactoring is in order here but it's an improvement. Thanks, Richard.
