Re: [PATCH] Fortran: degree trigonometric functions and procedure pointers [PR121366]

Mikael Morin <[email protected]> Mon, 6 Jul 2026 22:05:30 +0200
Newsgroups gmane.comp.gcc.patches,gmane.comp.gcc.fortran
Message-ID <[email protected]>
Le 05/07/2026 à 22:39, Harald Anlauf a écrit :
> [Argh, pressed send too early...]
> 
> Dear all,
> 
> this patch fills in a gap for the degree trigonometric functions: there
> is a library version with C calling conventions (by value), but the
> Fortran wrappers were missing.  For their use as procedure pointer
> target one needed to use the proper isym in resolution and generate
> the proper library name in line with our conventions.
> 
> The new wrappers were coded manually. 
The radian functions were made one way, and now the degree ones some 
other way.  That's a bit unfortunate, even if the maintenance burden 
should be rather low.  What didn't work with regenerate.sh and 
Makefile.am?  Was it the autoconf/automake version?  I can see that you 
managed to regenerate Makefile.in.

The rest looks good.  I understand little of it though.
Mikael

> I was not entirely sure if
> the kind=16/17 handling is fine, so feedback is appreciated.
> 
> The testcase checks both the F2023 variants as well as the existing
> GNU intrinsic extensions.  (Non-existing GNU extensions were removed
> from the documentation.)
> 
> I checked that the testcase passes locally also when specifying
> -freal-4-real-8, -freal-4-real-10, -freal-4-real-16,
> -freal-8-real-10, and -freal-8-real-16,
> but I chose to not test this explicitly in the testsuite.
> If it is felt important, it can be added.
> 
> Regtested cleanly on x86_64-pc-linux-gnu.  OK for mainline?
> 
> Do we want a backport to 16-branch?  If this is the case,
> I would need to adjust gfortran.map accordingly.
> 
> Thanks,
> Harald
>