[tree-ssa-loop] Remove redundant check for number of loops in pass_vectorize::execute
Tue Nov 9 07:17:34 GMT 2021
On Tue, Nov 9, 2021 at 7:34 AM Prathamesh Kulkarni
> On Mon, 8 Nov 2021 at 16:42, Richard Biener <email@example.com> wrote:
> > On Mon, Nov 8, 2021 at 12:06 PM Prathamesh Kulkarni via Gcc-patches
> > <firstname.lastname@example.org> wrote:
> > >
> > > Hi,
> > > The attached patch removes redundant check for number of loops in
> > > pass_vectorize::execute,
> > > since it only calls vectorize_loops, and in vectorize_loops, we
> > > immediately bail out if no loops are present:
> > > vect_loops_num = number_of_loops (cfun);
> > > /* Bail out if there are no loops. */
> > > if (vect_loops_num <= 1)
> > > return 0;
> > >
> > > Is the patch OK to commit ?
> > Can you please merge both functions then, keeping only
> > pass_vectorize::execute (and replacing 'cfun' occurrences
> > by 'fun' as passed as argument)?
> Done in the attached patch, does it look OK ?
> > Thanks,
> > Richard.
> > >
> > > Thanks,
> > > Prathamesh
More information about the Gcc-patches