[Bug target/89125] Misoptimization of converting sin(x) and cos(x) into sincos(x,&s,&c)

[Bug target/89125] Misoptimization of converting sin(x) and cos(x) into sincos(x,&s,&c)

[kargl at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125

kargl at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #45592|0                           |1
        is obsolete|                            |

--- Comment #13 from kargl at gcc dot gnu.org ---
Created attachment 52785
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52785&action=edit
New patch

New patch that deals with the *.c to *.cc rename

[Bug target/89125] Misoptimization of converting sin(x) and cos(x) into sincos(x,&s,&c)

[kargl at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125

--- Comment #14 from kargl at gcc dot gnu.org ---
*** Bug 105206 has been marked as a duplicate of this bug. ***

[Bug target/89125] Misoptimization of converting sin(x) and cos(x) into sincos(x,&s,&c)

[kargl at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125

kargl at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P4                          |P3

--- Comment #15 from kargl at gcc dot gnu.org ---
It's been over two years, can someone please commit the new path?

Testing shows 31 new passes and no regressions.

                === gcc Summary ===
                                w/o patch  w patch
# of expected passes            175405     175434
# of unexpected failures        1081       1051
# of unexpected successes       20         20
# of expected failures          1459       1459
# of unresolved testcases       10         10
# of unsupported tests          3252       3252

                === g++ Summary ===
                                w/o patch  w patch
# of expected passes            225338     225341
# of unexpected failures        678        676
# of expected failures          2071       2071
# of unresolved testcases       11         11
# of unsupported tests          10353      10353

                === gfortran Summary ===
                                w/o patch  w patch
# of expected passes            65901      65901
# of unexpected failures        12         12
# of expected failures          272        272
# of unsupported tests          100        100

[Bug target/89125] Misoptimization of converting sin(x) and cos(x) into sincos(x,&s,&c)

[kargl at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125

--- Comment #16 from kargl at gcc dot gnu.org ---
Can someone please commit the patch?

[Bug target/89125] Misoptimization of converting sin(x) and cos(x) into sincos(x,&s,&c)

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125

--- Comment #17 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Richard Biener <rguenth@gcc.gnu.org>:

https://gcc.gnu.org/g:b6e22db8564827c82108e0b7fa1a84675379c12b

commit r12-8240-gb6e22db8564827c82108e0b7fa1a84675379c12b
Author: Steve Kargl <kargl@gcc.gnu.org>
Date:   Mon Apr 25 09:23:56 2022 +0200

    target/89125 - BSD and math functions

    Back story: When GCC is configured and built on non-glibc platforms,
    it seems very little to no effort is made to enumerate the available
    C99 libm functions.  It is all or nothing for C99 libm.  The patch
    introduces a new function, used on only FreeBSD, to inform gcc that
    it has C99 libm functions (minus a few which clearly GCC does not check
    nor test).

    2022-04-15  Steven G. Kargl  <kargl@gcc.gnu.org>

            PR target/89125
            * config/freebsd.h: Define TARGET_LIBC_HAS_FUNCTION to be
            bsd_libc_has_function.
            * targhooks.cc (bsd_libc_has_function): New function.
            Expand the supported math functions to inclue C99 libm.
            * targhooks.h (bsd_libc_has_function): New Prototype.

[Bug target/89125] Misoptimization of converting sin(x) and cos(x) into sincos(x,&s,&c)

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED
      Known to work|                            |12.0

--- Comment #18 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed for GCC 12.

[Bug target/89125] Misoptimization of converting sin(x) and cos(x) into sincos(x,&s,&c)

[sgk at troutmask dot apl.washington.edu]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125

--- Comment #19 from Steve Kargl <sgk at troutmask dot apl.washington.edu> ---
On Mon, Apr 25, 2022 at 07:28:50AM +0000, rguenth at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125
> 
> Richard Biener <rguenth at gcc dot gnu.org> changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>          Resolution|---                         |FIXED
>              Status|NEW                         |RESOLVED
>       Known to work|                            |12.0
> 
> --- Comment #18 from Richard Biener <rguenth at gcc dot gnu.org> ---
> Fixed for GCC 12.
> 

Thanks.