This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch, fortran] PR32239 Use builtin_powi


:REVIEWMAIL:

Janne Blomqvist wrote:
> However, at the moment the vectorizer only vectorizes builtin_pow if
> the exponent is either 2 or 0.5, whereas if we expand constant
> exponents in the gfortran frontend (gfc_conv_cst_int_power) it
> vectorizes for other constant integer powers as well (the expansion by
> the frontend generates a number of calls to builtin_pow with exponent
> 2.0). So I left that part alone for now, i.e. builtin_powi is used
> only for real**int4 when int4 is non-constant.
This is now tracked as PR32503.

> So I guess at the moment this is mostly a library bloat reduction
> patch, with some potential for better optimization in the future.
Well, it might still help with the alias analysis etc.


Ok for the trunk. I assume you have regtested it; in any case it builds
and some limited testing shows no problems.

Tobias


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]