> 
> > 
> > > diff --git a/gcc/configure.ac b/gcc/configure.ac
> > > index d12382d2fa3..04503800423 100644
> > > --- a/gcc/configure.ac
> > > +++ b/gcc/configure.ac
> > > @@ -7894,6 +7894,21 @@ gif=`if test x$enable_x86_64_mfentry = xyes; then 
> > > echo 1; else echo 0; fi`
> > >   AC_DEFINE_UNQUOTED(ENABLE_X86_64_MFENTRY, $gif,
> > >   [Define to enable -mfentry by default on x86-64.])
> > > 
> > > +AFDO_PROFILER=perf
> > > +case "$cpu_type" in
> > > +  aarch64)
> > > +    AC_MSG_CHECKING(AArch64 build supports SPE profiling for 
> > > autoprofiledbootstrap)
> > > +    aarch64_spe_support=no
> > > +    if test -x /usr/bin/perf && test -n "$(/usr/bin/perf list | grep 
> > > arm_spe)"; then
> > > +      AFDO_PROFILER=perf_spe
> > 
> > But I'm actually surprised about this, I thought the standard perf
> > supported spe. Are you sure most users have a perf_spe?
> 
> I am not entirely sure what you mean. perf_spe is an argument being passed
> to create_gcov (--profiler=$(AFDO_PROFILER)). I guess the variable is
> confusingly named, though I couldn't come up with anything better.

Ah I thought AFDO_PROFILER was the perf command.  But of course it's in
gcc-auto-profile. Thanks for the clarification.

Looks all good to me with the changes.

-Andi

Reply via email to